Fix clicks on NTP tiles not contributing to Most Visited tiles

content/public/browser/content_browser_client.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 
 #include <stddef.h>
 
 #include <map>
@@ skipping 225 matching lines @@
   // that is allowed to commit.
   // This is called on the IO thread.
   virtual bool IsIllegalOrigin(ResourceContext* resource_context,
                                int child_process_id,
                                const GURL& origin);
 
   // Returns whether a URL should be allowed to open from a specific context.
   // 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,
                               const GURL& site_url);
 
   // Returns whether a new view for a new site instance can be added to a
   // given |process_host|.
   virtual bool MayReuseHost(RenderProcessHost* process_host);
 
   // Returns whether a new process should be created or an existing one should
@@ skipping 533 matching lines @@
   // Allows an embedder to provide its own ExternalVideoSurfaceContainer
   // implementation.  Return nullptr to disable external surface video.
   virtual ExternalVideoSurfaceContainer*
   OverrideCreateExternalVideoSurfaceContainer(WebContents* web_contents);
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_