Componentize GoogleURLTracker(InfoBarDelegate,MapEntry,NavHelper)

components/google/core/browser/google_url_tracker_client.h

Index: components/google/core/browser/google_url_tracker_client.h
diff --git a/components/google/core/browser/google_url_tracker_client.h b/components/google/core/browser/google_url_tracker_client.h
index 044a89abcac4e6bc6da5841748241273fd959a88..e22775561ecb8277d5bb6b036fa922d6f6533d66 100644
--- a/components/google/core/browser/google_url_tracker_client.h
+++ b/components/google/core/browser/google_url_tracker_client.h
@@ -6,6 +6,7 @@
 #define COMPONENTS_GOOGLE_GOOGLE_URL_TRACKER_CLIENT_H_
 
 #include "base/macros.h"
+#include "url/gurl.h"
 
 class GoogleURLTracker;
 class PrefService;
@@ -43,6 +44,18 @@ class GoogleURLTrackerClient {
   // should use.
   virtual net::URLRequestContextGetter* GetRequestContext() = 0;
 
+  // Returns whether |url| is for a Google domain.
+  // TODO(blundell): Eliminate this method in favor of having GoogleURLTracker
+  // call google_util::IsGoogleDomainURL() directly once google_util is
+  // componentized. crbug.com/381088
+  virtual bool IsGoogleDomainURL(const GURL& url) = 0;
+
+  // Appends the Google locale as a param to |url|.
+  // TODO(blundell): Eliminate this method in favor of having GoogleURLTracker

[Peter Kasting, 2014/06/06 18:05:39]

Nit: Technically, I think it's the infobar delegate that calls this method. 
Maybe these comments should both just say "callers call".

[blundell, 2014/06/06 18:49:02]

Done.

+  // call google_util::AppendGoogleLocaleParam() directly once google_util is
+  // componentized. crbug.com/381088
+  virtual GURL AppendGoogleLocaleParam(const GURL& url) = 0;
+
  protected:
   GoogleURLTracker* google_url_tracker() { return google_url_tracker_; }