Manually fetch favicons after hash changes.

ios/web/web_state/ui/crw_ui_web_view_web_controller.mm

Index: ios/web/web_state/ui/crw_ui_web_view_web_controller.mm
diff --git a/ios/web/web_state/ui/crw_ui_web_view_web_controller.mm b/ios/web/web_state/ui/crw_ui_web_view_web_controller.mm
index 5efb858d0e6f8282bb24ef10662775e5e9ee5150..75cd9f2c40390588c7b8811ad21441bc8847ecb9 100644
--- a/ios/web/web_state/ui/crw_ui_web_view_web_controller.mm
+++ b/ios/web/web_state/ui/crw_ui_web_view_web_controller.mm
@@ -821,6 +821,15 @@ const size_t kMaxMessageQueueSize = 262144;
                                    referrer:[self currentReferrer]];
   }
 
+  // Fetch the favicons for the new page.
+  // TODO(kkhorimoto): Consolidate favicon fetching into a single location.
+  // http://crbug.com/534573
+  NSString *const kGetFaviconsScript =
+       @"__gCrWeb.message.invokeOnHost("

[Eugene But (OOO till 7-30), 2015/09/21 21:32:43]

This approach is very fragile and will break if f.e. invokeOnHost will be
renamed. How about adding a public function to web JS?

[kkhorimoto, 2016/01/30 00:47:01]

Done.

+       "    {'command' : 'document.favicons',"
+       "     'favicons': __gCrWeb.common.getFavicons()});";
+  web::EvaluateJavaScript(_uiWebView, kGetFaviconsScript, nil);
+
   [super webPageChanged];
 }