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

Prevent browser exceptions from incorrectly triggering the assert in PDFPageProxy._abortOperatorList (PR 11069 follow-up)

Browse Source 
For certain canvas-related errors (and probably others), the browser rendering exceptions may be propagated "as-is" to the PDF.js code. In this case, the exceptions are of the somewhat cryptic `NS_ERROR_FAILURE` type.
Unfortunately these aren't actual `Error`s, which thus ends up unintentionally triggering the `assert` in `PDFPageProxy._abortOperatorList`; sorry about that!
This commit is contained in:
Jonas Jenwald 2019-11-07 11:37:48 +01:00
parent 4e0b02025f
commit 98f570c103
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/display/api.js
View File

@ -1378,7 +1378,8 @@ class PDFPageProxy {
* @private * @private
*/ */
_abortOperatorList({ intentState, reason, force = false, }) { _abortOperatorList({ intentState, reason, force = false, }) {
assert(reason instanceof Error, assert(reason instanceof Error ||
(typeof reason === 'object' && reason !== null),
'PDFPageProxy._abortOperatorList: Expected "reason" argument.'); 'PDFPageProxy._abortOperatorList: Expected "reason" argument.');
if (!intentState.streamReader) { if (!intentState.streamReader) {