Replace the DOMMatrix polyfill, used with Node.js, with the one fro…

…m `node-canvas` Fewer dependencies shouldn't be a bad idea in general, and given that the `node-canvas` package already include a `DOMMatrix` polyfill we can simply use that one instead.
mozilla · Sep 27, 2022 · c894be9 · c894be9
1 parent 5675a6e
commit c894be9
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 23 deletions.
diff --git a/gulpfile.js b/gulpfile.js
@@ -2169,7 +2169,7 @@ function packageJson() {
     bugs: DIST_BUGS_URL,
     license: DIST_LICENSE,
     dependencies: {
-      dommatrix: "^1.0.3",
+      canvas: "^2.10.1",
       "web-streams-polyfill": "^3.2.1",
     },
     browser: {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -14,7 +14,6 @@
     "core-js": "^3.25.1",
     "cross-env": "^7.0.3",
     "dialog-polyfill": "^0.5.6",
-    "dommatrix": "^1.0.3",
     "es-module-shims": "1.4.7",
     "eslint": "^8.23.1",
     "eslint-config-prettier": "^8.5.0",

diff --git a/src/shared/compatibility.js b/src/shared/compatibility.js
@@ -50,9 +50,7 @@ if (
     if (globalThis.DOMMatrix || !isNodeJS) {
       return;
     }
-    globalThis.DOMMatrix = __non_webpack_require__(
-      "dommatrix/dist/dommatrix.js"
-    );
+    globalThis.DOMMatrix = __non_webpack_require__("canvas").DOMMatrix;
   })();
 
   // Support: Node.js