Skip to content

Commit

Permalink
Merge pull request #12878 from Snuffleupagus/worker-compat-checks
Browse files Browse the repository at this point in the history
Remove redundant compatibility checks, for modern `generic` builds, in `src/core/worker.js`
  • Loading branch information
timvandermeij committed Jan 20, 2021
2 parents f45ba02 + b4eb552 commit 9d4bad9
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions src/core/worker.js
Original file line number Diff line number Diff line change
Expand Up @@ -139,19 +139,15 @@ class WorkerMessageHandler {

// Ensure that (primarily) Node.js users won't accidentally attempt to use
// a non-translated/non-polyfilled build of the library, since that would
// quickly fail anyway because of missing functionality (such as e.g.
// `ReadableStream` and `Promise.allSettled`).
// quickly fail anyway because of missing functionality.
if (
(typeof PDFJSDev === "undefined" || PDFJSDev.test("SKIP_BABEL")) &&
(typeof globalThis === "undefined" ||
typeof ReadableStream === "undefined" ||
typeof Promise.allSettled === "undefined")
typeof ReadableStream === "undefined"
) {
throw new Error(
"The browser/environment lacks native support for critical " +
"functionality used by the PDF.js library (e.g. `globalThis`, " +
"`ReadableStream`, and/or `Promise.allSettled`); " +
"please use an ES5-compatible build instead."
"functionality used by the PDF.js library (e.g. `ReadableStream`); " +
"please use an `es5`-build instead."
);
}
}
Expand Down

0 comments on commit 9d4bad9

Please sign in to comment.