Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(141)

Unified Diff: components/google/core/browser/google_url_tracker_client.h

Issue 316203003: Componentize GoogleURLTracker(InfoBarDelegate,MapEntry,NavHelper) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Nits Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..dc486866726170f36c88bcd4d5fb9d4e8af4ba3a 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 callers 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 callers 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_; }

Powered by Google App Engine
This is Rietveld 408576698