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

Merge pull request #12135 from Snuffleupagus/pdfManagerReady-loadDocument-promise

Browse Source 
[src/core/worker.js] Remove a useless Promise handler from the `pdfManagerReady` function
This commit is contained in:
Tim van der Meij 2020-07-29 00:07:25 +02:00 committed by GitHub
commit bcbbd03f7b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/core/worker.js
View File

@ -371,14 +371,12 @@ class WorkerMessageHandler {
function pdfManagerReady() { function pdfManagerReady() {
ensureNotTerminated(); ensureNotTerminated();
loadDocument(false).then( loadDocument(false).then(onSuccess, function (reason) {
onSuccess,
function loadFailure(ex) {
ensureNotTerminated(); ensureNotTerminated();
// Try again with recoveryMode == true // Try again with recoveryMode == true
if (!(ex instanceof XRefParseException)) { if (!(reason instanceof XRefParseException)) {
onFailure(ex); onFailure(reason);
return; return;
} }
pdfManager.requestLoadedStream(); pdfManager.requestLoadedStream();
@ -387,9 +385,7 @@ class WorkerMessageHandler {
loadDocument(true).then(onSuccess, onFailure); loadDocument(true).then(onSuccess, onFailure);
}); });
}, });
onFailure
);
} }
ensureNotTerminated(); ensureNotTerminated();