Fixes for the removal of setAndFetchFaviconForPage in https://bugzill…

…a.mozilla.org/show_bug.cgi?id=1894633
david-dick · Oct 13, 2024 · dd8f94b · dd8f94b
1 parent 880b078
commit dd8f94b
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/lib/Firefox/Marionette.pm b/lib/Firefox/Marionette.pm
@@ -2049,14 +2049,17 @@ let bookmarkStatus = (async function(bookmarkArguments) {
         );
       }
     } else {
-      let iconResult = placesUtils.PlacesUtils.favicons.setAndFetchFaviconForPage(
-        url,
-        rIconUrl,
-        true,
-        placesUtils.PlacesUtils.favicons.FAVICON_LOAD_NON_PRIVATE,
-        null,
-        Services.scriptSecurityManager.getSystemPrincipal()
-      );
+      if (placesUtils.PlacesUtils.favicons.setFaviconForPage) {
+      } else if (placesUtils.PlacesUtils.favicons.setAndFetchFaviconForPage) {
+        let iconResult = placesUtils.PlacesUtils.favicons.setAndFetchFaviconForPage(
+          url,
+          rIconUrl,
+          true,
+          placesUtils.PlacesUtils.favicons.FAVICON_LOAD_NON_PRIVATE,
+          null,
+          Services.scriptSecurityManager.getSystemPrincipal()
+        );
+      }
     }
   }
   return bookmark;