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

components/history/core/browser/history_service.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // The history system runs on a background thread so that potentially slow
 // database operations don't delay the browser. This backend processing is
 // represented by HistoryBackend. The HistoryService's job is to dispatch to
 // that thread.
 //
 // Main thread                       History thread
@@ skipping 100 matching lines @@
                               history_service_, init_status));
   }
 
   void SetInMemoryBackend(scoped_ptr<InMemoryHistoryBackend> backend) override {
     // Send the backend to the history service on the main thread.
     service_task_runner_->PostTask(
         FROM_HERE, base::Bind(&HistoryService::SetInMemoryBackend,
                               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,
                         const URLRow& row,
                         const RedirectList& redirects,
                         base::Time visit_time) override {
     service_task_runner_->PostTask(
         FROM_HERE,
         base::Bind(&HistoryService::NotifyURLVisited, history_service_,
                    transition, row, redirects, visit_time));
@@ skipping 968 matching lines @@
   FOR_EACH_OBSERVER(HistoryServiceObserver, observers_,
                     OnKeywordSearchTermUpdated(this, row, keyword_id, term));
 }
 
 void HistoryService::NotifyKeywordSearchTermDeleted(URLID url_id) {
   DCHECK(thread_checker_.CalledOnValidThread());
   FOR_EACH_OBSERVER(HistoryServiceObserver, observers_,
                     OnKeywordSearchTermDeleted(this, url_id));
 }
 
-scoped_ptr<base::CallbackList<void(const std::set<GURL>&)>::Subscription>
+scoped_ptr<base::CallbackList<void(const std::vector<GURL>&,
+                                   const std::vector<GURL>&)>::Subscription>
 HistoryService::AddFaviconChangedCallback(
     const HistoryService::OnFaviconChangedCallback& 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