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

Merge pull request #13822 from Snuffleupagus/ReadableStreams-cancel-no-Uncaught_promise

Browse Source 
Prevent "Uncaught promise" messages in the console when cancelling (some) `ReadableStream`s
This commit is contained in:
Tim van der Meij 2021-07-30 22:09:29 +02:00 committed by GitHub
commit 67f4c34f63
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/display/api.js
View File

@ -1772,7 +1772,11 @@ class PDFPageProxy {
return;
}
}
intentState.streamReader.cancel(new AbortException(reason?.message));
intentState.streamReader
.cancel(new AbortException(reason?.message))
.catch(() => {
// Avoid "Uncaught promise" messages in the console.
});
intentState.streamReader = null;
if (this._transport.destroyed) {

11
src/display/text_layer.js
View File

@ -597,7 +597,7 @@ class TextLayerRenderTask {
}
})
.catch(() => {
/* Avoid "Uncaught promise" messages in the console. */
// Avoid "Uncaught promise" messages in the console.
});
}
@ -615,7 +615,11 @@ class TextLayerRenderTask {
cancel() {
this._canceled = true;
if (this._reader) {
this._reader.cancel(new AbortException("TextLayer task cancelled."));
this._reader
.cancel(new AbortException("TextLayer task cancelled."))
.catch(() => {
// Avoid "Uncaught promise" messages in the console.
});
this._reader = null;
}
if (this._renderTimer !== null) {
@ -741,8 +745,7 @@ class TextLayerRenderTask {
pump();
} else {
throw new Error(
'Neither "textContent" nor "textContentStream"' +
" parameters specified."
'Neither "textContent" nor "textContentStream" parameters specified.'
);
}