Skip to content

Commit

Permalink
Remove the (unnecessary) WorkerMessageHandler variable from the `se…
Browse files Browse the repository at this point in the history
…tupFakeWorkerGlobal()` function in the `src/display/api.js` file
  • Loading branch information
Snuffleupagus committed Jan 31, 2018
1 parent 3f3992c commit f8d83b7
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions src/display/api.js
Original file line number Diff line number Diff line change
Expand Up @@ -1233,7 +1233,6 @@ var PDFWorker = (function PDFWorkerClosure() {

// Loads worker code into main thread.
function setupFakeWorkerGlobal() {
var WorkerMessageHandler;
if (fakeWorkerFilesLoadedCapability) {
return fakeWorkerFilesLoadedCapability.promise;
}
Expand All @@ -1245,26 +1244,24 @@ var PDFWorker = (function PDFWorkerClosure() {
fakeWorkerFilesLoadedCapability.resolve(mainWorkerMessageHandler);
return fakeWorkerFilesLoadedCapability.promise;
}
// In the developer build load worker_loader which in turn loads all the
// In the developer build load worker_loader.js which in turn loads all the
// other files and resolves the promise. In production only the
// pdf.worker.js file is needed.
if (typeof PDFJSDev === 'undefined' || !PDFJSDev.test('PRODUCTION')) {
if (typeof SystemJS === 'object') {
SystemJS.import('pdfjs/core/worker').then((worker) => {
WorkerMessageHandler = worker.WorkerMessageHandler;
fakeWorkerFilesLoadedCapability.resolve(WorkerMessageHandler);
fakeWorkerFilesLoadedCapability.resolve(worker.WorkerMessageHandler);
});
} else if (typeof require === 'function') {
var worker = require('../core/worker.js');
WorkerMessageHandler = worker.WorkerMessageHandler;
fakeWorkerFilesLoadedCapability.resolve(WorkerMessageHandler);
let worker = require('../core/worker.js');
fakeWorkerFilesLoadedCapability.resolve(worker.WorkerMessageHandler);
} else {
throw new Error(
'SystemJS or CommonJS must be used to load fake worker.');
}
} else {
var loader = fakeWorkerFilesLoader || function (callback) {
Util.loadScript(getWorkerSrc(), function () {
let loader = fakeWorkerFilesLoader || function(callback) {
Util.loadScript(getWorkerSrc(), function() {
callback(window.pdfjsDistBuildPdfWorker.WorkerMessageHandler);
});
};
Expand Down

0 comments on commit f8d83b7

Please sign in to comment.