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

components/bookmarks/browser/bookmark_model.h

Index: components/bookmarks/browser/bookmark_model.h
diff --git a/components/bookmarks/browser/bookmark_model.h b/components/bookmarks/browser/bookmark_model.h
index 50760c804766b570f7dd781a3955b118f4056f0e..a746cfb250a50f078e833fa1b220aec4746d6a15 100644
--- a/components/bookmarks/browser/bookmark_model.h
+++ b/components/bookmarks/browser/bookmark_model.h
@@ -166,6 +166,10 @@ class BookmarkModel : public KeyedService {
   // Returns the set of nodes with the |url|.
   void GetNodesByURL(const GURL& url, std::vector<const BookmarkNode*>* nodes);
 
+  // Returns the set of nodes which use the favicon at |icon_url|.
+  void GetNodesByIconURL(const GURL& icon_url,
+                         std::vector<const BookmarkNode*>* nodes);

[sky, 2015/06/19 16:38:02]

I don't think we need order here. How about a set?

+
   // Returns the most recently added user node for the |url|; urls from any
   // nodes that are not editable by the user are never returned by this call.
   // Returns NULL if |url| is not bookmarked.
@@ -298,7 +302,8 @@ class BookmarkModel : public KeyedService {
 
   // Notify BookmarkModel that the favicons for |urls| have changed and have to
   // be refetched. This notification is sent by BookmarkClient.
-  void OnFaviconChanged(const std::set<GURL>& urls);
+  void OnFaviconsChanged(const std::vector<GURL>& page_urls,
+                         const std::vector<GURL>& icon_urls);
 
   // Returns the client used by this BookmarkModel.
   BookmarkClient* client() const { return client_; }