Turn GoogleURLTrackerNavigationHelper(Impl) into a per-tab object.

chrome/browser/google/google_url_tracker_navigation_helper.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 CHROME_BROWSER_GOOGLE_GOOGLE_URL_TRACKER_NAVIGATION_HELPER_H_
 #define CHROME_BROWSER_GOOGLE_GOOGLE_URL_TRACKER_NAVIGATION_HELPER_H_
 
 class GoogleURLTracker;
-class InfoBarService;
-class Profile;
-
-namespace content {
-class NavigationController;
-}
 
 // A helper class for GoogleURLTracker that abstracts the details of listening
 // for various navigation events.
 class GoogleURLTrackerNavigationHelper {
  public:
+  GoogleURLTrackerNavigationHelper(GoogleURLTracker* tracker);

[Peter Kasting, 2014/05/15 21:29:52]

Nit: Explicit

[blundell, 2014/05/16 11:54:18]

Done.

   virtual ~GoogleURLTrackerNavigationHelper() {}

[Peter Kasting, 2014/05/15 21:29:52]

Nit: Put this destructor definition in the .cc file

[blundell, 2014/05/16 11:54:18]

Done.

 
-  // Sets the GoogleURLTracker that is associated with this object.
-  virtual void SetGoogleURLTracker(GoogleURLTracker* tracker) = 0;
+  // Enables or disables listening for navigation commits.
+  // OnNavigationCommitted will be called for each navigation commit if
+  // listening is enabled.
+  virtual void SetListeningForNavigationCommit(bool listen) = 0;
 
-  // Enables or disables listening for navigation commits for the given
-  // NavigationController. OnNavigationCommitted will be called for each
-  // navigation commit if listening is enabled.
-  virtual void SetListeningForNavigationCommit(
-      const content::NavigationController* nav_controller,
-      bool listen) = 0;
+  // Returns whether or not this object is currently listening for navigation
+  // commits.
+  virtual bool IsListeningForNavigationCommit() = 0;
 
-  // Returns whether or not the observer is currently listening for navigation
-  // commits for the given NavigationController.
-  virtual bool IsListeningForNavigationCommit(
-      const content::NavigationController* nav_controller) = 0;
+  // Enables or disables listening for tab destruction. OnTabClosed will be
+  // called on tab destruction if listening is enabled.
+  virtual void SetListeningForTabDestruction(bool listen) = 0;
 
-  // Enables or disables listening for tab destruction for the given
-  // NavigationController. OnTabClosed will be called on tab destruction if
-  // listening is enabled.
-  virtual void SetListeningForTabDestruction(
-      const content::NavigationController* nav_controller,
-      bool listen) = 0;
+  // Returns whether or not this object is currently listening for tab
+  // destruction.
+  virtual bool IsListeningForTabDestruction() = 0;
 
-  // Returns whether or not the observer is currently listening for tab
-  // destruction for the given NavigationController.
-  virtual bool IsListeningForTabDestruction(
-      const content::NavigationController* nav_controller) = 0;
+ protected:
+  GoogleURLTracker* google_url_tracker() { return tracker_; }

[Peter Kasting, 2014/05/15 21:29:52]

Nit: See comments on other CL regarding accessor/member name harmonization

[blundell, 2014/05/16 11:54:18]

Done.

+
+ private:
+  GoogleURLTracker* tracker_;
 };

[Peter Kasting, 2014/05/15 21:29:52]

Nit: DISALLOW_COPY_AND_ASSIGN

[blundell, 2014/05/16 11:54:18]

Done.

 
 #endif  // CHROME_BROWSER_GOOGLE_GOOGLE_URL_TRACKER_NAVIGATION_HELPER_H_