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

Ensure that PDFScriptingManager.setDocument handles failure when initializing the scripting-factory

Browse Source 
This way, we'll immediately clean-up in exactly the same way as the other failure code-paths in the `PDFScriptingManager.setDocument` method.
This commit is contained in:
Jonas Jenwald 2021-06-17 18:42:15 +02:00
parent 6dc5dd194f
commit 5db7a3cc88
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
web/pdf_scripting_manager.js
View File

@ -96,7 +96,14 @@ class PDFScriptingManager {
if (pdfDocument !== this._pdfDocument) { if (pdfDocument !== this._pdfDocument) {
return; // The document was closed while the data resolved. return; // The document was closed while the data resolved.
} }
this._scripting = this._createScripting(); try {
this._scripting = this._createScripting();
} catch (error) {
console.error(`PDFScriptingManager.setDocument: "${error?.message}".`);
await this._destroyScripting();
return;
}
this._internalEvents.set("updatefromsandbox", event => { this._internalEvents.set("updatefromsandbox", event => {
if (event?.source !== window) { if (event?.source !== window) {