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

Remove the ability to pass in more than one font to the _prepareFontLoadEvent method (PR 10539 follow-up)

Browse Source 
After the changes in PR 10539 (which landed over three years ago) the `FontLoader.bind` method can only be called with *a single* font at a time, hence the `_prepareFontLoadEvent` method obviously don't need to support multiple fonts any more.
This commit is contained in:
Jonas Jenwald 2022-09-26 13:25:26 +02:00
parent ca05b650aa
commit 2161f334a0
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/display/font_loader.js
View File

@ -114,7 +114,7 @@ class FontLoader {
}
await new Promise(resolve => {
const request = this._queueLoadingCallback(resolve);
this._prepareFontLoadEvent([font], request);
this._prepareFontLoadEvent(font, request);
});
// The font was, asynchronously, loaded.
}
@ -218,7 +218,7 @@ class FontLoader {
return shadow(this, "_loadTestFont", testFont);
}
_prepareFontLoadEvent(fonts, request) {
_prepareFontLoadEvent(font, request) {
if (typeof PDFJSDev !== "undefined" && PDFJSDev.test("MOZCENTRAL")) {
throw new Error("Not implemented: _prepareFontLoadEvent");
}
@ -299,19 +299,13 @@ class FontLoader {
const rule = `@font-face {font-family:"${loadTestFontId}";src:${url}}`;
this.insertRule(rule);
const names = [];
for (const font of fonts) {
names.push(font.loadedName);
}
names.push(loadTestFontId);
const div = this._document.createElement("div");
div.style.visibility = "hidden";
div.style.width = div.style.height = "10px";
div.style.position = "absolute";
div.style.top = div.style.left = "0px";
for (const name of names) {
for (const name of [font.loadedName, loadTestFontId]) {
const span = this._document.createElement("span");
span.textContent = "Hi";
span.style.fontFamily = name;