From fcfb2457bc4c7862102d9939f164d0b08b27931c Mon Sep 17 00:00:00 2001
From: Calixte Denizet <calixte.denizet@gmail.com>
Date: Thu, 3 Aug 2023 09:46:27 +0200
Subject: [PATCH] [Editor] Use a getter and not a function for the
 _checkIfSVGFitsInCanvas (bug 1846727)

It's a followup of #16780.
---
 src/display/editor/tools.js | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/display/editor/tools.js b/src/display/editor/tools.js
index efa4ce834..98c44bd56 100644
--- a/src/display/editor/tools.js
+++ b/src/display/editor/tools.js
@@ -75,7 +75,7 @@ class ImageManager {
 
   #cache = null;
 
-  static _checkIfSVGFitsInCanvas() {
+  static get _isSVGFittingCanvas() {
     // By default, Firefox doesn't rescale without preserving the aspect ratio
     // when drawing an SVG image on a canvas, see https://bugzilla.mozilla.org/1547776.
     // The "workaround" is to append "svgView(preserveAspectRatio(none))" to the
@@ -91,7 +91,7 @@ class ImageManager {
       return new Uint32Array(ctx.getImageData(0, 0, 1, 1).data.buffer)[0] === 0;
     });
 
-    return shadow(this, "_checkIfSVGFitsInCanvas", promise);
+    return shadow(this, "_isSVGFittingCanvas", promise);
   }
 
   async #get(key, rawData) {
@@ -128,8 +128,7 @@ class ImageManager {
       if (image.type === "image/svg+xml") {
         // Unfortunately, createImageBitmap doesn't work with SVG images.
         // (see https://bugzilla.mozilla.org/1841972).
-        const mustRemoveAspectRatioPromise =
-          ImageManager._checkIfSVGFitsInCanvas();
+        const mustRemoveAspectRatioPromise = ImageManager._isSVGFittingCanvas;
         const fileReader = new FileReader();
         const imageElement = new Image();
         const imagePromise = new Promise((resolve, reject) => {