From 6fa46499f01a96e8dd9cecfa1ebc9f6cf7e26b60 Mon Sep 17 00:00:00 2001 From: treib Date: Wed, 12 Aug 2015 09:27:34 -0700 Subject: [PATCH] Popular sites on the NTP: Favicon improvements - Check for existing favicons by page URL rather than icon URL to better handle cases where the icon URL changed etc. BUG=516618 Review URL: https://codereview.chromium.org/1287993002 Cr-Commit-Position: refs/heads/master@{#343030} --- chrome/browser/android/favicon_helper.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/chrome/browser/android/favicon_helper.cc b/chrome/browser/android/favicon_helper.cc index 50783ef7770b..7f008f274cc3 100644 --- a/chrome/browser/android/favicon_helper.cc +++ b/chrome/browser/android/favicon_helper.cc @@ -287,8 +287,8 @@ void FaviconHelper::EnsureFaviconIsAvailable( profile, web_contents, page_url, favicon_url); favicon::FaviconService* service = FaviconServiceFactory::GetForProfile( profile, ServiceAccessType::IMPLICIT_ACCESS); - service->GetFaviconImage(favicon_url, callback_runner, - cancelable_task_tracker_.get()); + service->GetFaviconImageForPageURL(page_url, callback_runner, + cancelable_task_tracker_.get()); } FaviconHelper::~FaviconHelper() {} -- 2.11.4.GIT