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

Don't initialize L10n.#elements eagerly since it's unused in MOZCENTRAL builds

Browse Source 
It's not necessary to manually start translation in the Firefox PDF Viewer, and doing so would even cause problems there (see issue 17142).
This commit is contained in:
Jonas Jenwald 2024-10-30 15:20:44 +01:00
parent f013c39b9f
commit cdd4b052f9
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
web/l10n.js
View File

@ -23,7 +23,7 @@
class L10n {
#dir;
#elements = new Set();
#elements;
#lang;
@ -71,7 +71,7 @@ class L10n {
/** @inheritdoc */
async translate(element) {
this.#elements.add(element);
(this.#elements ||= new Set()).add(element);
try {
this.#l10n.connectRoot(element);
await this.#l10n.translateRoots();
@ -91,10 +91,13 @@ class L10n {
/** @inheritdoc */
async destroy() {
for (const element of this.#elements) {
this.#l10n.disconnectRoot(element);
if (this.#elements) {
for (const element of this.#elements) {
this.#l10n.disconnectRoot(element);
}
this.#elements.clear();
this.#elements = null;
}
this.#elements.clear();
this.#l10n.pauseObserving();
}