Merge pull request #12476 from Snuffleupagus/worker-rm-handler-scope

Remove the `scope` parameter from the "GetOperatorList" handler in `src/core/worker.js` (PR 11110 follow-up)
This commit is contained in:
Tim van der Meij 2020-10-13 21:08:48 +02:00 committed by GitHub
commit 0700a3de36
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -599,9 +599,7 @@ class WorkerMessageHandler {
}); });
}); });
handler.on( handler.on("GetOperatorList", function wphSetupRenderPage(data, sink) {
"GetOperatorList",
function wphSetupRenderPage(data, sink) {
var pageIndex = data.pageIndex; var pageIndex = data.pageIndex;
pdfManager.getPage(pageIndex).then(function (page) { pdfManager.getPage(pageIndex).then(function (page) {
var task = new WorkerTask(`GetOperatorList: page ${pageIndex}`); var task = new WorkerTask(`GetOperatorList: page ${pageIndex}`);
@ -645,15 +643,12 @@ class WorkerMessageHandler {
sink.error(reason); sink.error(reason);
// TODO: Should `reason` be re-thrown here (currently that // TODO: Should `reason` be re-thrown here (currently that casues
// casues "Uncaught exception: ..." messages in the // "Uncaught exception: ..." messages in the console)?
// console)?
} }
); );
}); });
}, });
this
);
handler.on("GetTextContent", function wphExtractText(data, sink) { handler.on("GetTextContent", function wphExtractText(data, sink) {
var pageIndex = data.pageIndex; var pageIndex = data.pageIndex;