Fix clicks on NTP tiles not contributing to Most Visited tiles

content/browser/frame_host/navigator_impl.cc

Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index c63d436dd916555d18149a188b5df67d3b9ca79c..b854bbaaf096734c5d7535da807da2ae3ca9e1a6 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -722,6 +722,9 @@ void NavigatorImpl::RequestOpenURL(
     params.is_renderer_initiated = false;
   }
 
+  GetContentClient()->browser()->OverrideOpenURLParams(current_site_instance,
+                                                       &params);

[Charlie Reis, 2016/06/28 00:52:05]

This seems a little unfortunate since we call right back out to the embedder
with the params on the line below.  I suppose the params immutable in that call,
though, so perhaps we need this.

[mastiz, 2016/06/28 08:20:31]

We could make a modified copy of params within RequestOpenURL but it seems more
convoluted to me: this alternative is rather readable and consistent with the
equivalent logic for web-UI.

+
   if (delegate_)
     delegate_->RequestOpenURL(render_frame_host, params);
 }