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

Unified Diff: chrome/browser/sync/sessions/sessions_sync_manager.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, 5 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/sessions_sync_manager.h
diff --git a/chrome/browser/sync/sessions/sessions_sync_manager.h b/chrome/browser/sync/sessions/sessions_sync_manager.h
index 24db41110b978f77db8fe96ee267687d54b3ba13..1e5fd22a75ec3d0dfdaeab8b42ca7597fede8d0e 100644
--- a/chrome/browser/sync/sessions/sessions_sync_manager.h
+++ b/chrome/browser/sync/sessions/sessions_sync_manager.h
@@ -63,12 +63,13 @@ class LocalSessionEventHandler {
virtual void OnLocalTabModified(SyncedTabDelegate* modified_tab) = 0;
// A local navigation occurred that triggered updates to favicon data for
- // each URL in |updated_page_urls|. This is routed through Sessions Sync so
- // that we can filter (exclude) favicon updates for pages that aren't
- // currently part of the set of local open tabs, and pass relevant updates
- // on to FaviconCache for out-of-band favicon syncing.
- virtual void OnFaviconPageUrlsUpdated(
- const std::set<GURL>& updated_page_urls) = 0;
+ // each page URL in |page_urls| (e.g. http://www.google.com) and the icon URL
+ // |icon_url| (e.g. http://www.google.com/favicon.ico). This is routed through
+ // Sessions Sync so that we can filter (exclude) favicon updates for pages
+ // that aren't currently part of the set of local open tabs, and pass relevant
+ // updates on to FaviconCache for out-of-band favicon syncing.
+ virtual void OnFaviconsChanged(const std::set<GURL>& page_urls,
+ const GURL& icon_url) = 0;
};
// The LocalSessionEventRouter is responsible for hooking itself up to various
@@ -122,8 +123,8 @@ class SessionsSyncManager : public syncer::SyncableService,
// LocalSessionEventHandler implementation.
void OnLocalTabModified(SyncedTabDelegate* modified_tab) override;
- void OnFaviconPageUrlsUpdated(
- const std::set<GURL>& updated_favicon_page_urls) override;
+ void OnFaviconsChanged(const std::set<GURL>& page_urls,
+ const GURL& icon_url) override;
// Returns the tag used to uniquely identify this machine's session in the
// sync model.

Powered by Google App Engine
This is Rietveld 408576698