Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix the remaining Promise.resolve(undefined) cases

Browse Source 
Many years ago now there were some `Promise` implementations that had issues resolving with an *implicitly* `undefined` value. That should no longer be the case, and we've not been using the `Promise.resolve(undefined)` format for a long time, hence this patch fixes the few remaining cases.
This commit is contained in:
Jonas Jenwald 2021-10-03 15:32:33 +02:00
parent a37bc609e8
commit cd22c31752
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
web/pdf_page_view.js
View File

@ -831,7 +831,7 @@ class PDFPageView {
renderTask.promise.then( renderTask.promise.then(
function () { function () {
showCanvas(); showCanvas();
renderCapability.resolve(undefined); renderCapability.resolve();
}, },
function (error) { function (error) {
showCanvas(); showCanvas();

2
web/pdf_thumbnail_view.js
View File

@ -278,7 +278,7 @@ class PDFThumbnailView {
draw() { draw() {
if (this.renderingState !== RenderingStates.INITIAL) { if (this.renderingState !== RenderingStates.INITIAL) {
console.error("Must be in new state before drawing"); console.error("Must be in new state before drawing");
return Promise.resolve(undefined); return Promise.resolve();
} }
const { pdfPage } = this; const { pdfPage } = this;