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

chrome/browser/sync/test/integration/sessions_helper.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.
 
 #include "chrome/browser/sync/test/integration/sessions_helper.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 126 matching lines @@
   }
 
   void OnLocalTabModified(
       browser_sync::SyncedTabDelegate* modified_tab) override {
     // Unwind to ensure SessionsSyncManager has processed the event.
     base::MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&TabEventHandler::QuitLoop, weak_factory_.GetWeakPtr()));
   }
 
-  void OnFaviconPageUrlsUpdated(
-      const std::set<GURL>& updated_page_urls) override {
+  void OnFaviconsChanged(const std::vector<GURL>& page_urls,
+                         const std::vector<GURL>& icon_urls) override {

[Roger McFarlane (Chromium), 2015/05/13 16:07:05]

nit: comment out the param names, as they're not used in the body.

[pkotwicz, 2015/05/13 17:26:49]

Done.

     // Unwind to ensure SessionsSyncManager has processed the event.
     base::MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&TabEventHandler::QuitLoop, weak_factory_.GetWeakPtr()));
   }
 
  private:
   void QuitLoop() {
     base::MessageLoop::current()->Quit();
   }
@@ skipping 244 matching lines @@
   return !checker.TimedOut();
 }
 
 void DeleteForeignSession(int index, std::string session_tag) {
   ProfileSyncServiceFactory::GetInstance()->GetForProfile(
       test()->GetProfile(index))->
           GetOpenTabsUIDelegate()->DeleteForeignSession(session_tag);
 }
 
 }  // namespace sessions_helper