Remove default-page favicon for interstitials

chrome/browser/favicon/favicon_utils.cc

Index: chrome/browser/favicon/favicon_utils.cc
diff --git a/chrome/browser/favicon/favicon_utils.cc b/chrome/browser/favicon/favicon_utils.cc
index 18c9bc53d9da0e9dc0b677a560e0c7cca278b708..f44f5a3aeb1e42910027012487264947588a7d4d 100644
--- a/chrome/browser/favicon/favicon_utils.cc
+++ b/chrome/browser/favicon/favicon_utils.cc
@@ -46,6 +46,11 @@ void CreateContentFaviconDriverForWebContents(
 }
 
 bool ShouldDisplayFavicon(content::WebContents* web_contents) {
+  // No favicon on interstitials. This check must be done first since
+  // interstitial navigations don't commit and always have a pending entry.
+  if (web_contents->ShowingInterstitialPage())
+    return false;
+
   // Always display a throbber during pending loads.
   const content::NavigationController& controller =
       web_contents->GetController();