Remove NOTIFICATION_HISTORY_URL_VISITED

chrome/browser/ui/cocoa/history_menu_bridge.h

Index: chrome/browser/ui/cocoa/history_menu_bridge.h
diff --git a/chrome/browser/ui/cocoa/history_menu_bridge.h b/chrome/browser/ui/cocoa/history_menu_bridge.h
index 088ac24a03a7edd6444e8dd656825b30eb6a5a11..39638a2337dee5fc6de1674ebe9868239ad3744b 100644
--- a/chrome/browser/ui/cocoa/history_menu_bridge.h
+++ b/chrome/browser/ui/cocoa/history_menu_bridge.h
@@ -17,6 +17,7 @@
 #include "chrome/browser/sessions/tab_restore_service.h"
 #include "chrome/browser/sessions/tab_restore_service_observer.h"
 #import "chrome/browser/ui/cocoa/main_menu_item.h"
+#include "components/history/core/browser/history_service_observer.h"
 #include "components/sessions/session_id.h"
 #include "content/public/browser/notification_observer.h"
 
@@ -58,7 +59,8 @@ struct FaviconImageResult;
 // class does the bulk of the work.
 class HistoryMenuBridge : public content::NotificationObserver,
                           public TabRestoreServiceObserver,
-                          public MainMenuItem {
+                          public MainMenuItem,
+                          public history::HistoryServiceObserver {
  public:
   // This is a generalization of the data we store in the history menu because
   // we pull things from different sources with different data types.
@@ -131,6 +133,13 @@ class HistoryMenuBridge : public content::NotificationObserver,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) override;
 
+  // history::HistoryServiceObserver:

[Peter Kasting, 2014/10/17 20:41:03]

Nit: Please place overrides for classes in the same order as the classes are
declared in lines 60-63

+  virtual void OnURLVisited(HistoryService* history_service,
+                            ui::PageTransition transition,
+                            const history::URLRow& row,
+                            const history::RedirectList& redirects,
+                            base::Time visit_time) override;
+
   // TabRestoreServiceObserver:
   virtual void TabRestoreServiceChanged(TabRestoreService* service) override;
   virtual void TabRestoreServiceDestroyed(TabRestoreService* service) override;
@@ -176,6 +185,9 @@ class HistoryMenuBridge : public content::NotificationObserver,
   // Does the query for the history information to create the menu.
   void CreateMenu();
 
+  // Invoked when the History information has changed.
+  void OnHistoryChanged();
+
   // Callback method for when HistoryService query results are ready with the
   // most recently-visited sites.
   void OnVisitedHistoryResults(history::QueryResults* results);