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

Merge pull request #22059 from Snuffleupagus/FontInfo-rm-extra

Browse Source 
Stop passing `extra` properties to the `FontInfo` class
This commit is contained in:
Tim van der Meij
2026-10-03 12:51:53 +02:00
committed by GitHub
parent b5809f83d6 02aa7f5e91
commit 87a106c034
3 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/display/api.js
+1 -1
View File
@@ -2787,7 +2787,7 @@ class WorkerTransport {
break;
}
const fontData = new FontInfo(exportedData);
const fontData = new FontInfo(exportedData.buffer);
const inspectFont =
this._params.pdfBug && globalThis.FontInspector?.enabled
? (font, url) => globalThis.FontInspector.fontAdded(font, url)
src/display/obj_bin_transform_display.js
+1 -4
View File
@@ -110,12 +110,9 @@ class FontInfo {
#view;
constructor({ buffer, extra }) {
constructor(buffer) {
this.#buffer = buffer;
this.#view = new DataView(buffer);
if (extra) {
Object.assign(this, extra);
}
}
#readBoolean(index) {
test/unit/obj_bin_transform_spec.js
+2 -2
View File
@@ -129,7 +129,7 @@ describe("obj_bin_transform", function () {
sizeEstimate += 4 + fontInfo.data.length;
const buffer = compileFontInfo(fontInfo);
expect(buffer.byteLength).toEqual(sizeEstimate);
const deserialized = new FontInfo({ buffer });
const deserialized = new FontInfo(buffer);
expect(deserialized.black).toBeTrue();
expect(deserialized.bold).toBeTrue();
expect(deserialized.disableFontFace).toBeTrue();
@@ -160,7 +160,7 @@ describe("obj_bin_transform", function () {
cssFontInfo,
systemFontInfo,
});
const deserialized = new FontInfo({ buffer });
const deserialized = new FontInfo(buffer);
expect(deserialized.cssFontInfo.fontWeight).toEqual("not a number");
expect(deserialized.systemFontInfo.src).toEqual("source");
});