DidStopLoadingIcons should account for remote main frame.

BUG=399775

Review URL: https://codereview.chromium.org/634123002

Cr-Commit-Position: refs/heads/master@{#299318}

content/renderer/render_view_impl.cc

@@ -4178,6 +4178,11 @@ void RenderViewImpl::DidStopLoadingIcons() {
   if (TouchEnabled())
     icon_types |= WebIconURL::TypeTouchPrecomposed | WebIconURL::TypeTouch;
 
+  // Favicons matter only for the top-level frame. If it is a WebRemoteFrame,
+  // just return early.
+  if (webview()->mainFrame()->isWebRemoteFrame())
+    return;
+
   WebVector<WebIconURL> icon_urls =
       webview()->mainFrame()->iconURLs(icon_types);