Bug 1253930 - Recently bookmarked items have the default favicon instead of the correct one. r=dao draft
authorMarco Bonardo <mbonardo@mozilla.com>
Mon, 07 Mar 2016 12:06:40 +0100
changeset 337347 66a025372127b9ad70dac23cac63ca70eed000a2
parent 337346 d41587d6dd40ec34e7823d54b288c2d0adf43133
child 515641 9b4c9879a6fb24055b162c798e495009afae3ecd
push id12333
push usermak77@bonardo.net
push dateMon, 07 Mar 2016 11:07:33 +0000
reviewersdao
bugs1253930
milestone47.0a1
Bug 1253930 - Recently bookmarked items have the default favicon instead of the correct one. r=dao MozReview-Commit-ID: LMTVQetQL9Q
browser/base/content/browser-places.js
--- a/browser/base/content/browser-places.js
+++ b/browser/base/content/browser-places.js
@@ -1398,18 +1398,17 @@ var BookmarkingUI = {
         document.createElementNS("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul",
                                  "menuitem");
       item.setAttribute("label", title || uri);
       item.setAttribute("targetURI", uri);
       item.setAttribute("class", "menuitem-iconic menuitem-with-favicon bookmark-item " +
                                  extraCSSClass);
       item.addEventListener("command", onItemCommand);
       if (icon) {
-        let iconURL = "moz-anno:favicon:" + icon;
-        item.setAttribute("image", iconURL);
+        item.setAttribute("image", icon);
       }
       fragment.appendChild(item);
     }
     root.containerOpen = false;
     aHeaderItem.parentNode.insertBefore(fragment, aHeaderItem.nextSibling);
   },
 
   /**