From e89afa589929e31575de83f21f2d4810cdc94291 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Tue, 5 Jun 2018 20:29:56 +0200 Subject: [PATCH] Stop sending the `PDFManagerReady` message from the Worker, since it's unused in the API After PR 8617 the `PDFManagerReady` message handler function, in `src/display/api.js`, is now a no-op. Hence it seems completely unnecessary to keep sending this message from `src/core/worker.js`. --- src/core/worker.js | 3 +-- src/display/api.js | 3 --- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/src/core/worker.js b/src/core/worker.js index 567343864..106d31bcb 100644 --- a/src/core/worker.js +++ b/src/core/worker.js @@ -629,9 +629,8 @@ var WorkerMessageHandler = { newPdfManager.terminate(); throw new Error('Worker was terminated'); } - pdfManager = newPdfManager; - handler.send('PDFManagerReady', null); + pdfManager.onLoadedStream().then(function(stream) { handler.send('DataLoaded', { length: stream.bytes.byteLength, }); }); diff --git a/src/display/api.js b/src/display/api.js index 3bd0a60ee..a9415b071 100644 --- a/src/display/api.js +++ b/src/display/api.js @@ -1834,9 +1834,6 @@ var WorkerTransport = (function WorkerTransportClosure() { this.downloadInfoCapability.resolve(data); }, this); - messageHandler.on('PDFManagerReady', function transportPage(data) { - }, this); - messageHandler.on('StartRenderPage', function transportRender(data) { if (this.destroyed) { return; // Ignore any pending requests if the worker was terminated.