diff --git a/web/base_viewer.js b/web/base_viewer.js index 428e1c8cf..7db0d193f 100644 --- a/web/base_viewer.js +++ b/web/base_viewer.js @@ -1467,12 +1467,9 @@ class BaseViewer { } cleanup() { - for (let i = 0, ii = this._pages.length; i < ii; i++) { - if ( - this._pages[i] && - this._pages[i].renderingState !== RenderingStates.FINISHED - ) { - this._pages[i].reset(); + for (const pageView of this._pages) { + if (pageView.renderingState !== RenderingStates.FINISHED) { + pageView.reset(); } } } @@ -1481,10 +1478,8 @@ class BaseViewer { * @private */ _cancelRendering() { - for (let i = 0, ii = this._pages.length; i < ii; i++) { - if (this._pages[i]) { - this._pages[i].cancelRendering(); - } + for (const pageView of this._pages) { + pageView.cancelRendering(); } } @@ -1873,8 +1868,8 @@ class BaseViewer { viewer.textContent = ""; if (this._spreadMode === SpreadMode.NONE) { - for (let i = 0, ii = pages.length; i < ii; ++i) { - viewer.appendChild(pages[i].div); + for (const pageView of this._pages) { + viewer.appendChild(pageView.div); } } else { const parity = this._spreadMode - 1; diff --git a/web/pdf_thumbnail_viewer.js b/web/pdf_thumbnail_viewer.js index d7e6221c1..f0ffd3c7a 100644 --- a/web/pdf_thumbnail_viewer.js +++ b/web/pdf_thumbnail_viewer.js @@ -151,12 +151,9 @@ class PDFThumbnailViewer { } cleanup() { - for (let i = 0, ii = this._thumbnails.length; i < ii; i++) { - if ( - this._thumbnails[i] && - this._thumbnails[i].renderingState !== RenderingStates.FINISHED - ) { - this._thumbnails[i].reset(); + for (const thumbnail of this._thumbnails) { + if (thumbnail.renderingState !== RenderingStates.FINISHED) { + thumbnail.reset(); } } TempImageFactory.destroyCanvas(); @@ -237,10 +234,8 @@ class PDFThumbnailViewer { * @private */ _cancelRendering() { - for (let i = 0, ii = this._thumbnails.length; i < ii; i++) { - if (this._thumbnails[i]) { - this._thumbnails[i].cancelRendering(); - } + for (const thumbnail of this._thumbnails) { + thumbnail.cancelRendering(); } }