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

Merge pull request #18103 from Snuffleupagus/pr-18089-followup

Browse Source 
Actually disable `TextLayerRenderTask.prototype.#processItems` when `MAX_TEXT_DIVS_TO_RENDER` is reached (PR 18089 follow-up)
This commit is contained in:
Tim van der Meij 2024-05-16 14:17:09 +02:00 committed by GitHub
commit 128705c16e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/display/text_layer.js
View File

@ -192,6 +192,8 @@ function layout(params) {
} }
class TextLayerRenderTask { class TextLayerRenderTask {
#disableProcessItems = false;
#reader = null; #reader = null;
#textContentSource = null; #textContentSource = null;
@ -278,6 +280,9 @@ class TextLayerRenderTask {
} }
#processItems(items, lang) { #processItems(items, lang) {
if (this.#disableProcessItems) {
return;
}
if (!this._layoutTextParams.ctx) { if (!this._layoutTextParams.ctx) {
this._textDivProperties.set(this._rootContainer, { lang }); this._textDivProperties.set(this._rootContainer, { lang });
this._layoutTextParams.ctx = getCtx(lang); this._layoutTextParams.ctx = getCtx(lang);
@ -291,7 +296,7 @@ class TextLayerRenderTask {
if (textDivs.length > MAX_TEXT_DIVS_TO_RENDER) { if (textDivs.length > MAX_TEXT_DIVS_TO_RENDER) {
warn("Ignoring additional textDivs for performance reasons."); warn("Ignoring additional textDivs for performance reasons.");
this._processItems = () => {}; // Avoid multiple warnings for one page. this.#disableProcessItems = true; // Avoid multiple warnings for one page.
return; return;
} }