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

Unified Diff: chrome/browser/google/google_url_tracker.h

Issue 9811022: Misc. small cleanups to minimize TemplateURL refactoring diffs: (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 9 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: chrome/browser/google/google_url_tracker.h
===================================================================
--- chrome/browser/google/google_url_tracker.h (revision 128075)
+++ chrome/browser/google/google_url_tracker.h (working copy)
@@ -54,9 +54,7 @@
// Only the main browser process loop should call this, when setting up
// g_browser_process->google_url_tracker_. No code other than the
// GoogleURLTracker itself should actually use
- // g_browser_process->google_url_tracker() (which shouldn't be hard, since
- // there aren't useful public functions on this object for consumers to access
- // anyway).
+ // g_browser_process->google_url_tracker().
explicit GoogleURLTracker(Mode mode);
virtual ~GoogleURLTracker();
@@ -91,7 +89,7 @@
friend class GoogleURLTrackerInfoBarDelegate;
friend class GoogleURLTrackerTest;
- typedef InfoBarDelegate* (*InfobarCreator)(InfoBarTabHelper*,
+ typedef InfoBarDelegate* (*InfoBarCreator)(InfoBarTabHelper*,
GoogleURLTracker*,
const GURL&);
@@ -132,7 +130,7 @@
void ShowGoogleURLInfoBarIfNecessary(content::WebContents* web_contents);
content::NotificationRegistrar registrar_;
- InfobarCreator infobar_creator_;
+ InfoBarCreator infobar_creator_;
// TODO(ukai): GoogleURLTracker should track google domain (e.g. google.co.uk)
// rather than URL (e.g. http://www.google.co.uk/), so that user could
// configure to use https in search engine templates.

Powered by Google App Engine
This is Rietveld 408576698