Bug 1434922 - Set a favicon for about:privatebrowsing. r=florian draft
authorJohann Hofmann <jhofmann@mozilla.com>
Sun, 04 Feb 2018 11:52:23 +0100
changeset 750979 05745b3e4b911407875fc50e5a9a3f1a67fbafd5
parent 750946 92b6195a9367dec27fbf3efbe7824cf163aa017a
push id97812
push userjhofmann@mozilla.com
push dateSun, 04 Feb 2018 11:20:45 +0000
reviewersflorian
bugs1434922
milestone60.0a1
Bug 1434922 - Set a favicon for about:privatebrowsing. r=florian This was initially removed when we added support for setting the favicon ahead of parsing the page for newtab-like pages. Turns out we need it in some scenarios, such as navigating back and forth or duplicating the tab. MozReview-Commit-ID: HuMeEnqpLLB
browser/components/privatebrowsing/content/aboutPrivateBrowsing.xhtml
--- a/browser/components/privatebrowsing/content/aboutPrivateBrowsing.xhtml
+++ b/browser/components/privatebrowsing/content/aboutPrivateBrowsing.xhtml
@@ -14,17 +14,17 @@
   <!ENTITY % browserDTD SYSTEM "chrome://browser/locale/browser.dtd">
   %browserDTD;
   <!ENTITY % aboutPrivateBrowsingDTD SYSTEM "chrome://browser/locale/aboutPrivateBrowsing.dtd">
   %aboutPrivateBrowsingDTD;
 ]>
 
 <html xmlns="http://www.w3.org/1999/xhtml" class="private">
   <head>
-    <link id="favicon" rel="icon" type="image/png"/>
+    <link id="favicon" rel="icon" type="image/png" href="chrome://browser/skin/privatebrowsing/favicon.svg"/>
     <link rel="stylesheet" href="chrome://browser/content/aboutPrivateBrowsing.css" type="text/css" media="all"/>
     <link rel="stylesheet" href="chrome://browser/skin/privatebrowsing/aboutPrivateBrowsing.css" type="text/css" media="all"/>
     <script type="application/javascript" src="chrome://browser/content/aboutPrivateBrowsing.js"></script>
   </head>
 
   <body dir="&locale.dir;">
     <p class="showNormal">&aboutPrivateBrowsing.notPrivate;</p>
     <button xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"