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

chrome/browser/google/chrome_google_url_tracker_client.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/google/chrome_google_url_tracker_client.h"
 
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/google/google_url_tracker.h"
+#include "chrome/browser/google/google_url_tracker_navigation_helper_impl.h"
 #include "chrome/browser/infobars/infobar_service.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/web_contents.h"
 
 ChromeGoogleURLTrackerClient::ChromeGoogleURLTrackerClient() {
 }
 
 ChromeGoogleURLTrackerClient::~ChromeGoogleURLTrackerClient() {
 }
 
-void ChromeGoogleURLTrackerClient::SetListeningForNavigationStart(
-    bool listen) {
+void ChromeGoogleURLTrackerClient::SetListeningForNavigationStart(bool listen) {
   if (listen) {
-    registrar_.Add(this, content::NOTIFICATION_NAV_ENTRY_PENDING,
+    registrar_.Add(
+        this,
+        content::NOTIFICATION_NAV_ENTRY_PENDING,
         content::NotificationService::AllBrowserContextsAndSources());
   } else {
-    registrar_.Remove(this, content::NOTIFICATION_NAV_ENTRY_PENDING,
+    registrar_.Remove(
+        this,
+        content::NOTIFICATION_NAV_ENTRY_PENDING,
         content::NotificationService::AllBrowserContextsAndSources());
   }
 }
 
 bool ChromeGoogleURLTrackerClient::IsListeningForNavigationStart() {
-  return registrar_.IsRegistered(this, content::NOTIFICATION_NAV_ENTRY_PENDING,
+  return registrar_.IsRegistered(
+      this,
+      content::NOTIFICATION_NAV_ENTRY_PENDING,
       content::NotificationService::AllBrowserContextsAndSources());
 }
 
 void ChromeGoogleURLTrackerClient::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   DCHECK_EQ(content::NOTIFICATION_NAV_ENTRY_PENDING, type);
   content::NavigationController* controller =
       content::Source<content::NavigationController>(source).ptr();
+  GoogleURLTrackerNavigationHelperImpl* nav_helper =
+      GoogleURLTrackerNavigationHelperImpl::FromWebContents(
+          controller->GetWebContents());
   InfoBarService* infobar_service =
       InfoBarService::FromWebContents(controller->GetWebContents());
   // Because we're listening to all sources, there may be no
-  // InfoBarService for some notifications, e.g. navigations in
+  // InfoBarService/NavigationHelper for some notifications, e.g. navigations in

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

Super-tiny nit: Reverse order of infobar service/nav helper in both this comment
and the conditional, to match the declaration order above and the param order in
the call below.

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

Irrelevant now.

On 2014/05/15 21:29:52, Peter Kasting wrote:

   // bubbles/balloons etc.
-  if (infobar_service) {
+  if (infobar_service && nav_helper) {
     google_url_tracker()->OnNavigationPending(
-        controller,
+        nav_helper,
         infobar_service,
         controller->GetPendingEntry()->GetUniqueID());
   }
 }