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

Actually disable TextLayerRenderTask.prototype.#processItems when MAX_TEXT_DIVS_TO_RENDER is reached (PR 18089 follow-up)

Browse Source 
I broke this accidentally in PR 18089, sorry about that!
Note that since `#processItems` is private we can no longer just "replace" the method as was done in PR 18052.
This commit is contained in:
Jonas Jenwald 2024-05-16 11:42:04 +02:00
parent 4db843617f
commit d5f3829f91
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;
} }