Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-08-06 14:27:23 +02:00
Code Issues Packages Projects Releases Wiki Activity

Move SCALE_MATRIX into CanvasGraphics and initialize its DOMMatrix lazily (issue 21720)

Browse Source 
While this shouldn't affect performance negatively, it does however (ever so slightly) increase the bundle size of the Firefox PDF Viewer.
If that's not desirable, since it's only needed in order to improve Node.js compatibility, we could also just WONTFIX the issue.
This commit is contained in:
Jonas Jenwald 2026-08-06 13:21:16 +02:00
parent 1b4055e958
commit ac5aa273f2
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/display/canvas.js
View File

@ -66,10 +66,6 @@ const EXECUTION_STEPS = 10;
const FULL_CHUNK_HEIGHT = 16; const FULL_CHUNK_HEIGHT = 16;
// Only used in rescaleAndStroke. The goal is to avoid
// creating a new DOMMatrix object each time we need it.
const SCALE_MATRIX = new DOMMatrix();
// Used to get some coordinates. // Used to get some coordinates.
const XY = new Float32Array(2); const XY = new Float32Array(2);
@ -496,6 +492,10 @@ const NORMAL_CLIP = {};
const EO_CLIP = {}; const EO_CLIP = {};
class CanvasGraphics { class CanvasGraphics {
// Only used in rescaleAndStroke. The goal is to avoid
// creating a new DOMMatrix object each time we need it.
static #SCALE_MATRIX = null;
// Knockout group support fields. // Knockout group support fields.
#knockoutGroupLevel = 0; #knockoutGroupLevel = 0;
@ -4320,6 +4320,7 @@ class CanvasGraphics {
ctx.stroke(path); ctx.stroke(path);
return; return;
} }
const SCALE_MATRIX = (CanvasGraphics.#SCALE_MATRIX ??= new DOMMatrix());
const dashes = ctx.getLineDash(); const dashes = ctx.getLineDash();
if (saveRestore) { if (saveRestore) {