Remove NOTIFICATION_HISTORY_URL_VISITED

chrome/browser/extensions/api/history/history_api.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_EXTENSIONS_API_HISTORY_HISTORY_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_HISTORY_HISTORY_API_H_
 
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "chrome/browser/extensions/chrome_extension_function.h"
 #include "chrome/browser/history/history_notifications.h"
 #include "chrome/browser/history/history_service.h"
 #include "chrome/common/extensions/api/history.h"
+#include "components/history/core/browser/history_service_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "extensions/browser/browser_context_keyed_api_factory.h"
 #include "extensions/browser/event_router.h"
 
 namespace base {
 class ListValue;
 }
 
 namespace extensions {
 
 // Observes History service and routes the notifications as events to the
 // extension system.
-class HistoryEventRouter : public content::NotificationObserver {
+class HistoryEventRouter : public content::NotificationObserver,
+                           public history::HistoryServiceObserver {
  public:
-  explicit HistoryEventRouter(Profile* profile);
+  explicit HistoryEventRouter(Profile* profile,

[not at google - send to devlin, 2014/10/14 16:06:41]

explicit no longer needed.

+                              HistoryService* history_service);
   virtual ~HistoryEventRouter();
 
  private:
   // content::NotificationObserver::Observe.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) override;
 
-  void HistoryUrlVisited(Profile* profile,
-                         const history::URLVisitedDetails* details);
+  // history::HistoryServiceObserver.
+  virtual void OnURLVisited(HistoryService* history_service,
+                            ui::PageTransition transition,
+                            const history::URLRow& row,
+                            const history::RedirectList& redirects,
+                            base::Time visit_time) override;
 
   void HistoryUrlsRemoved(Profile* profile,
                           const history::URLsDeletedDetails* details);
 
   void DispatchEvent(Profile* profile,
                      const std::string& event_name,
                      scoped_ptr<base::ListValue> event_args);
 
   // Used for tracking registrations to history service notifications.
   content::NotificationRegistrar registrar_;
+  Profile* profile_;
+  HistoryService* history_service_;
 
   DISALLOW_COPY_AND_ASSIGN(HistoryEventRouter);
 };
 
 class HistoryAPI : public BrowserContextKeyedAPI, public EventRouter::Observer {
  public:
   explicit HistoryAPI(content::BrowserContext* context);
   virtual ~HistoryAPI();
 
   // KeyedService implementation.
@@ skipping 138 matching lines @@
   // HistoryFunctionWithCallback:
   virtual bool RunAsyncImpl() override;
 
   // Callback for the history service to acknowledge deletion.
   void DeleteComplete();
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_HISTORY_HISTORY_API_H_