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

Reduce duplication when handling the "test" message from the worker

Browse Source 
The feature-testing on the worker-thread has been simplified in previous pull requests, which means that we can simplify this main-thread handler as well.
This commit is contained in:
Jonas Jenwald 2024-06-15 13:25:51 +02:00
parent 0a36b667e4
commit 8d4456172b
1 changed files with 11 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/display/api.js
View File

@ -2173,32 +2173,26 @@ class PDFWorker {
messageHandler.on("test", data => { messageHandler.on("test", data => {
worker.removeEventListener("error", onWorkerError); worker.removeEventListener("error", onWorkerError);
if (this.destroyed) { if (this.destroyed || !data) {
terminateEarly(); terminateEarly();
return; // worker was destroyed return;
} }
if (data) { this._messageHandler = messageHandler;
this._messageHandler = messageHandler; this._port = worker;
this._port = worker; this._webWorker = worker;
this._webWorker = worker;
this._readyCapability.resolve(); this._readyCapability.resolve();
// Send global setting, e.g. verbosity level. // Send global setting, e.g. verbosity level.
messageHandler.send("configure", { messageHandler.send("configure", {
verbosity: this.verbosity, verbosity: this.verbosity,
}); });
} else {
this._setupFakeWorker();
messageHandler.destroy();
worker.terminate();
}
}); });
messageHandler.on("ready", data => { messageHandler.on("ready", data => {
worker.removeEventListener("error", onWorkerError); worker.removeEventListener("error", onWorkerError);
if (this.destroyed) { if (this.destroyed) {
terminateEarly(); terminateEarly();
return; // worker was destroyed return;
} }
try { try {
sendTest(); sendTest();