Index: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc |
diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc |
index 931e44a2c6464a1410b3d8a13f3736ea178c2b0d..af2e8197bfd31aa6df9a07a38daafda4369c4d72 100644 |
--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc |
+++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc |
@@ -578,7 +578,7 @@ void RecentTabsSubMenuModel::AddDeviceFavicon( |
case browser_sync::SyncedSession::TYPE_UNSET: |
favicon_id = IDR_LAPTOP_FAVICON; |
break; |
- }; |
+ } |
ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance(); |
SetIcon(index_in_menu, rb.GetNativeImageNamed(favicon_id)); |
@@ -613,8 +613,8 @@ void RecentTabsSubMenuModel::AddTabFavicon(int command_id, const GURL& url) { |
if (!favicon_service) |
return; |
- favicon_service->GetFaviconImageForURL( |
- FaviconService::FaviconForURLParams( |
+ favicon_service->GetFaviconImageForPageURL( |
+ FaviconService::FaviconForPageURLParams( |
url, favicon_base::FAVICON, gfx::kFaviconSize), |
base::Bind(&RecentTabsSubMenuModel::OnFaviconDataAvailable, |
weak_ptr_factory_.GetWeakPtr(), |
@@ -653,7 +653,8 @@ void RecentTabsSubMenuModel::ClearLocalEntries() { |
while (last_local_model_index_ >= 0) |
RemoveItemAt(last_local_model_index_--); |
- // Cancel asynchronous FaviconService::GetFaviconImageForURL() tasks of all |
+ // Cancel asynchronous FaviconService::GetFaviconImageForPageURL() tasks of |
+ // all |
// local tabs. |
local_tab_cancelable_task_tracker_.TryCancelAll(); |