Refactor HistoryService to not send NOTIFICATION_FAVICON_CHANGED

chrome/browser/sync/sessions/notification_service_sessions_router.h

Index: chrome/browser/sync/sessions/notification_service_sessions_router.h
diff --git a/chrome/browser/sync/sessions/notification_service_sessions_router.h b/chrome/browser/sync/sessions/notification_service_sessions_router.h
index 7ce1c144c67c3c4632c52d599617a6b85eb203f2..b049eb3d631f259ccf63d15a7ccfb77c84008b71 100644
--- a/chrome/browser/sync/sessions/notification_service_sessions_router.h
+++ b/chrome/browser/sync/sessions/notification_service_sessions_router.h
@@ -5,11 +5,16 @@
 #ifndef CHROME_BROWSER_SYNC_SESSIONS_NOTIFICATION_SERVICE_SESSIONS_ROUTER_H_
 #define CHROME_BROWSER_SYNC_SESSIONS_NOTIFICATION_SERVICE_SESSIONS_ROUTER_H_
 
+#include <set>
+
+#include "base/callback_list.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/sync/sessions/sessions_sync_manager.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
+class GURL;
+class HistoryService;
 class Profile;
 
 namespace content {
@@ -47,12 +52,20 @@ class NotificationServiceSessionsRouter
   // from WebContents.
   void OnNavigationBlocked(content::WebContents* web_contents);
 
+  // Called when the urls of favicon changed.
+  void OnFaviconChanged(HistoryService* history_service,
+                        const std::set<GURL>& changed_favicons);
+
   LocalSessionEventHandler* handler_;
   content::NotificationRegistrar registrar_;
   Profile* const profile_;
   syncer::SyncableService::StartSyncFlare flare_;
   base::WeakPtrFactory<NotificationServiceSessionsRouter> weak_ptr_factory_;
 
+  scoped_ptr<base::CallbackList<
+      void(HistoryService*, const std::set<GURL>&)>::Subscription>
+      favicon_changed_subscription_;
+
   DISALLOW_COPY_AND_ASSIGN(NotificationServiceSessionsRouter);
 };