Bookmark updates for https://bugzilla.mozilla.org/show_bug.cgi?id=155…

…2815
david-dick · May 4, 2024 · c8792fb · c8792fb
1 parent 029ff6d
commit c8792fb
Showing 1 changed file with 14 additions and 5 deletions.
diff --git a/lib/Firefox/Marionette.pm b/lib/Firefox/Marionette.pm
@@ -1897,18 +1897,27 @@ let bookmarkStatus = (async function(bookmarkArguments) {
     let rIconUrl = netUtil.NetUtil.newURI(iconUrl);
     if (bookmarkArguments["icon"]) {
       let icon = bookmarkArguments["icon"];
-      placesUtils.PlacesUtils.favicons.replaceFaviconDataFromDataURL(
-        rIconUrl,
-        icon,
-      );
-      let iconResult = placesUtils.PlacesUtils.favicons.setAndFetchFaviconForPage(
+      if (placesUtils.PlacesUtils.favicons.setFaviconForPage) {
+        let iconDataUrl = netUtil.NetUtil.newURI(icon);
+        placesUtils.PlacesUtils.favicons.setFaviconForPage(
+          url,
+          rIconUrl,
+          iconDataUrl
+        );
+      } else {
+        placesUtils.PlacesUtils.favicons.replaceFaviconDataFromDataURL(
+          rIconUrl,
+          icon
+        );
+        let iconResult = placesUtils.PlacesUtils.favicons.setAndFetchFaviconForPage(
           url,
           rIconUrl,
           false,
           placesUtils.PlacesUtils.favicons.FAVICON_LOAD_NON_PRIVATE,
           null,
           Services.scriptSecurityManager.getSystemPrincipal()
         );
+      }
     } else {
       let iconResult = placesUtils.PlacesUtils.favicons.setAndFetchFaviconForPage(
         url,