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

Slightly reduce the size of the FontInspector-integration in the API

Browse Source 
Given that this functionality only applies in the viewer, when `PDFBug` is being enabled and used, it can't hurt to slightly reduce the size of this code.
This commit is contained in:
Jonas Jenwald 2023-03-23 14:02:02 +01:00
parent 184f5701e9
commit 378caa7203
2 changed files with 9 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/display/api.js
View File

@ -2762,19 +2762,15 @@ class WorkerTransport {
break;
}
let fontRegistry = null;
if (params.pdfBug && globalThis.FontInspector?.enabled) {
fontRegistry = {
registerFont(font, url) {
globalThis.FontInspector.fontAdded(font, url);
},
};
}
const inspectFont =
params.pdfBug && globalThis.FontInspector?.enabled
? (font, url) => globalThis.FontInspector.fontAdded(font, url)
: null;
const font = new FontFaceObject(exportedData, {
isEvalSupported: params.isEvalSupported,
disableFontFace: params.disableFontFace,
ignoreErrors: params.ignoreErrors,
fontRegistry,
inspectFont,
});
this.fontLoader

8
src/display/font_loader.js
View File

@ -322,7 +322,7 @@ class FontFaceObject {
isEvalSupported = true,
disableFontFace = false,
ignoreErrors = false,
fontRegistry = null,
inspectFont = null,
}
) {
this.compiledGlyphs = Object.create(null);
@ -333,7 +333,7 @@ class FontFaceObject {
this.isEvalSupported = isEvalSupported !== false;
this.disableFontFace = disableFontFace === true;
this.ignoreErrors = ignoreErrors === true;
this.fontRegistry = fontRegistry;
this._inspectFont = inspectFont;
}
createNativeFontFace() {
@ -357,7 +357,7 @@ class FontFaceObject {
);
}
this.fontRegistry?.registerFont(this);
this._inspectFont?.(this);
return nativeFontFace;
}
@ -379,7 +379,7 @@ class FontFaceObject {
rule = `@font-face {font-family:"${this.cssFontInfo.fontFamily}";${css}src:${url}}`;
}
this.fontRegistry?.registerFont(this, url);
this._inspectFont?.(this, url);
return rule;
}