Bug 523369: [Regression] Secure favicon.ico request pop-ups certificate error dialog [r=vingtetun]
--- a/mobile/chrome/content/browser-ui.js
+++ b/mobile/chrome/content/browser-ui.js
@@ -385,20 +385,16 @@ var BrowserUI = {
ExtensionsView.uninit();
ConsoleView.uninit();
},
update : function(aState) {
let icons = document.getElementById("urlbar-icons");
let browser = Browser.selectedBrowser;
- // Use documentURIObject in the favicon construction so that we
- // do the right thing with about:-style error pages. Bug 515188
- let uri = browser.contentDocument.documentURIObject;
-
switch (aState) {
case TOOLBARSTATE_LOADED:
if (icons.getAttribute("mode") != "edit")
icons.setAttribute("mode", "view");
this._updateIcon(browser.mIconURL);
break;
--- a/mobile/chrome/content/browser.js
+++ b/mobile/chrome/content/browser.js
@@ -2594,17 +2594,19 @@ Tab.prototype = {
}
catch (e) {
faviconURI = null;
}
}
if (!faviconURI || faviconURI.schemeIs("javascript") || gFaviconService.isFailedFavicon(faviconURI)) {
try {
- faviconURI = gIOService.newURI(this._browser.currentURI.prePath + "/favicon.ico", null, null);
+ // Use documentURIObject in the favicon construction so that we
+ // do the right thing with about:-style error pages. Bug 515188
+ faviconURI = gIOService.newURI(this._browser.contentDocument.documentURIObject.prePath + "/favicon.ico", null, null);
gFaviconService.setAndLoadFaviconForPage(this._browser.currentURI, faviconURI, true);
}
catch (e) {
faviconURI = null;
}
if (faviconURI && gFaviconService.isFailedFavicon(faviconURI))
faviconURI = null;
}