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

[api-minor] Remove the, in legacy builds, bundled DOMMatrix polyfill

Browse Source 
According to the MDN compatibility data, see https://developer.mozilla.org/en-US/docs/Web/API/DOMMatrix/DOMMatrix#browser_compatibility, all browsers that we support have native `DOMMatrix` implementations (since quite some time too).

Hence Node.js is the only environment that lack `DOMMatrix` support, which probably isn't that surprising given that it's browser functionality.
While the `DOMMatrix` polyfill isn't that large, it nonetheless seems completely unnecessary to bundle it in the `legacy` builds when it's not needed in browsers. However, we can avoid that by simply listing `dommatrix` as a dependency for the `pdfjs-dist` library.
This commit is contained in:
Jonas Jenwald 2022-03-08 10:03:40 +01:00
parent 3e593cfc1d
commit 157a71d404
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
gulpfile.js
View File

@ -2043,6 +2043,7 @@ function packageBowerJson() {
bugs: DIST_BUGS_URL, bugs: DIST_BUGS_URL,
license: DIST_LICENSE, license: DIST_LICENSE,
dependencies: { dependencies: {
dommatrix: "^0.0.24",
"web-streams-polyfill": "^3.2.0", "web-streams-polyfill": "^3.2.0",
}, },
peerDependencies: { peerDependencies: {

4
src/shared/compatibility.js
View File

@ -50,7 +50,9 @@ if (
if (globalThis.DOMMatrix || !isNodeJS) { if (globalThis.DOMMatrix || !isNodeJS) {
return; return;
} }
globalThis.DOMMatrix = require("dommatrix/dist/dommatrix.js"); globalThis.DOMMatrix = __non_webpack_require__(
"dommatrix/dist/dommatrix.js"
);
})(); })();
// Provides support for *recent* additions to the Promise specification, // Provides support for *recent* additions to the Promise specification,