Fix clicks on NTP tiles not contributing to Most Visited tiles

content/public/browser/content_browser_client.h

Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index 231bd2edb06b2a4a1c8bdfd5f3cfebead1734862..d46ef21af68f3e3c874e4a9cc83761dcabad809e 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -243,6 +243,10 @@ class CONTENT_EXPORT ContentBrowserClient {
   // This also applies in cases where the new URL will open in another process.
   virtual bool ShouldAllowOpenURL(SiteInstance* site_instance, const GURL& url);
 
+  // Allows the embedder to override OpenURLParams.
+  virtual void OverrideOpenURLParams(const SiteInstance* site_instance,

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

nit: No const.

http://www.chromium.org/developers/content-module/content-api
"don't add the const identifier to interfaces. For interfaces implemented by the
embedder, we can't make assumptions about what the embedder needs to implement
it. For interfaces implemented by content, the implementation details doesn't
have to be exposed."

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

Done.

+                                     OpenURLParams* params) {}
+
   // Returns whether a new view for a given |site_url| can be launched in a
   // given |process_host|.
   virtual bool IsSuitableHost(RenderProcessHost* process_host,