Update all bookmarks which use an icon URL when a favicon's bitmap is updated

components/history/core/browser/history_service.cc

Index: components/history/core/browser/history_service.cc
diff --git a/components/history/core/browser/history_service.cc b/components/history/core/browser/history_service.cc
index 23fdad993c2f1737c65631162535565c7346ae7d..aede20579bdc47b5f21b756b6231474753c5e40d 100644
--- a/components/history/core/browser/history_service.cc
+++ b/components/history/core/browser/history_service.cc
@@ -118,11 +118,12 @@ class HistoryService::BackendDelegate : public HistoryBackend::Delegate {
                               history_service_, base::Passed(&backend)));
   }
 
-  void NotifyFaviconChanged(const std::set<GURL>& urls) override {
+  void NotifyFaviconsChanged(const std::vector<GURL>& page_urls,
+                             const std::vector<GURL>& icon_urls) override {
     // Send the notification to the history service on the main thread.
     service_task_runner_->PostTask(
-        FROM_HERE, base::Bind(&HistoryService::NotifyFaviconChanged,
-                              history_service_, urls));
+        FROM_HERE, base::Bind(&HistoryService::NotifyFaviconsChanged,
+                              history_service_, page_urls, icon_urls));
   }
 
   void NotifyURLVisited(ui::PageTransition transition,
@@ -1112,17 +1113,18 @@ void HistoryService::NotifyKeywordSearchTermDeleted(URLID url_id) {
                     OnKeywordSearchTermDeleted(this, url_id));
 }
 
-scoped_ptr<base::CallbackList<void(const std::set<GURL>&)>::Subscription>
-HistoryService::AddFaviconChangedCallback(
-    const HistoryService::OnFaviconChangedCallback& callback) {
+scoped_ptr<base::CallbackList<void(const std::vector<GURL>&,
+                                   const std::vector<GURL>&)>::Subscription>
+HistoryService::AddFaviconsChangedCallback(
+    const HistoryService::OnFaviconsChangedCallback& callback) {
   DCHECK(thread_checker_.CalledOnValidThread());
   return favicon_changed_callback_list_.Add(callback);
 }
 
-void HistoryService::NotifyFaviconChanged(
-    const std::set<GURL>& changed_favicons) {
+void HistoryService::NotifyFaviconsChanged(const std::vector<GURL>& page_urls,
+                                           const std::vector<GURL>& icon_urls) {
   DCHECK(thread_checker_.CalledOnValidThread());
-  favicon_changed_callback_list_.Notify(changed_favicons);
+  favicon_changed_callback_list_.Notify(page_urls, icon_urls);
 }
 
 }  // namespace history