Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-04-16 10:14:03 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #20836 from Snuffleupagus/FontFaceObject-fix-asserts

Browse Source 
Fix the `disableFontFace` and `fontExtraProperties` asserts in the `FontFaceObject` constructor (PR 20197 follow-up)
This commit is contained in:
Jonas Jenwald 2026-03-09 20:08:58 +01:00 committed by GitHub
commit 873378b718
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 13 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/display/font_loader.js
View File

@ -355,19 +355,22 @@ class FontLoader {
}
class FontFaceObject {
compiledGlyphs = Object.create(null);
#fontData;
constructor(translatedData, inspectFont = null, extra, charProcOperatorList) {
this.compiledGlyphs = Object.create(null);
this.#fontData = translatedData;
if (typeof PDFJSDev === "undefined" || PDFJSDev.test("TESTING")) {
if (typeof this.disableFontFace !== "boolean") {
unreachable("disableFontFace must be available.");
}
if (typeof this.fontExtraProperties !== "boolean") {
unreachable("fontExtraProperties must be available.");
}
assert(
typeof translatedData.disableFontFace === "boolean",
"disableFontFace must be available."
);
assert(
typeof translatedData.fontExtraProperties === "boolean",
"fontExtraProperties must be available."
);
}
this.#fontData = translatedData;
this._inspectFont = inspectFont;
if (extra) {
Object.assign(this, extra);
@ -453,7 +456,7 @@ class FontFaceObject {
}
get disableFontFace() {
return this.#fontData.disableFontFace ?? false;
return this.#fontData.disableFontFace;
}
set disableFontFace(value) {
@ -461,7 +464,7 @@ class FontFaceObject {
}
get fontExtraProperties() {
return this.#fontData.fontExtraProperties ?? false;
return this.#fontData.fontExtraProperties;
}
get isInvalidPDFjsFont() {