Redoing Issue 36073011: Allowing file:/// in Instant Extended's Most Visited links.

chrome/renderer/searchbox/searchbox.cc

Index: chrome/renderer/searchbox/searchbox.cc
diff --git a/chrome/renderer/searchbox/searchbox.cc b/chrome/renderer/searchbox/searchbox.cc
index 893b3dfd516477207314525dd4f37b3bd60dab57..baf906ba2d3b05d19328deaa0d5e87982239d992 100644
--- a/chrome/renderer/searchbox/searchbox.cc
+++ b/chrome/renderer/searchbox/searchbox.cc
@@ -167,6 +167,12 @@ void SearchBox::LogImpression(int position, const base::string16& provider) {
       provider));
 }
 
+void SearchBox::LogNavigation(int position, const base::string16& provider) {

[samarth, 2014/03/06 21:01:22]

I think LogMostVisitedClick is clearer here (and in SearchIPCRoucter, etc).

Can you also rename LogImpression to LogMostVistedImpression to match while
you're at it?

[huangs, 2014/03/07 00:33:41]

Done, but using LogMostVisitedNavigation() (will change if you insist).

+  render_view()->Send(new ChromeViewHostMsg_LogNavigation(
+      render_view()->GetRoutingID(), render_view()->GetPageId(), position,
+      provider));
+}
+
 void SearchBox::CheckIsUserSignedInToChromeAs(const base::string16& identity) {
   render_view()->Send(new ChromeViewHostMsg_ChromeIdentityCheck(
       render_view()->GetRoutingID(), render_view()->GetPageId(), identity));