Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(340)

Unified Diff: chrome/browser/sync/sessions/notification_service_sessions_router.h

Issue 1133463005: Update all bookmarks which use an icon URL when a favicon's bitmap is updated (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@startup_do_not_unexpire
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 81a775cbf5a74db6607cfc61131cc74a985146a9..e78127ec0b56b9e8217e37bcf6d8757f4b436ac4 100644
--- a/chrome/browser/sync/sessions/notification_service_sessions_router.h
+++ b/chrome/browser/sync/sessions/notification_service_sessions_router.h
@@ -51,15 +51,19 @@ class NotificationServiceSessionsRouter
// from WebContents.
void OnNavigationBlocked(content::WebContents* web_contents);
- // Called when the urls of favicon changed.
- void OnFaviconChanged(const std::set<GURL>& changed_favicons);
+ // Called when the favicons for the given page URLs
+ // (e.g. http://www.google.com) and the given icon URLs (e.g.
+ // http://www.google.com/favicon.ico) have changed.
+ void OnFaviconsChanged(const std::vector<GURL>& page_urls,
+ const std::vector<GURL>& icon_urls);
LocalSessionEventHandler* handler_;
content::NotificationRegistrar registrar_;
Profile* const profile_;
syncer::SyncableService::StartSyncFlare flare_;
- scoped_ptr<base::CallbackList<void(const std::set<GURL>&)>::Subscription>
+ scoped_ptr<base::CallbackList<void(const std::vector<GURL>&,
+ const std::vector<GURL>&)>::Subscription>
favicon_changed_subscription_;
base::WeakPtrFactory<NotificationServiceSessionsRouter> weak_ptr_factory_;

Powered by Google App Engine
This is Rietveld 408576698