Merge pull request #19206 from nicolo-ribaudo/canvas-prepend

Simplify logic to insert canvas as first element
mozilla · Dec 10, 2024 · aa589b9 · aa589b9
2 parents 44421d3 + 62db66d
commit aa589b9
Showing 1 changed file with 2 additions and 12 deletions.
diff --git a/web/pdf_page_view.js b/web/pdf_page_view.js
@@ -989,12 +989,7 @@ class PDFPageView {
         // drawing is complete when `!this.renderingQueue`, to prevent black
         // flickering.
         // In whatever case, the canvas must be the first child.
-        const { firstChild } = canvasWrapper;
-        if (firstChild) {
-          firstChild.before(canvas);
-        } else {
-          canvasWrapper.append(canvas);
-        }
+        canvasWrapper.prepend(canvas);
         showCanvas = null;
         return;
       }
@@ -1006,12 +1001,7 @@ class PDFPageView {
         prevCanvas.replaceWith(canvas);
         prevCanvas.width = prevCanvas.height = 0;
       } else {
-        const { firstChild } = canvasWrapper;
-        if (firstChild) {
-          firstChild.before(canvas);
-        } else {
-          canvasWrapper.append(canvas);
-        }
+        canvasWrapper.prepend(canvas);
       }
 
       showCanvas = null;