| Index: chrome/browser/favicon/favicon_tab_helper.cc
|
| diff --git a/chrome/browser/favicon/favicon_tab_helper.cc b/chrome/browser/favicon/favicon_tab_helper.cc
|
| index 912c6369a5a914a6481ad2aa858a74604ec16ffe..939b049e5653d34e2dc86a4624a25fb241bc2153 100644
|
| --- a/chrome/browser/favicon/favicon_tab_helper.cc
|
| +++ b/chrome/browser/favicon/favicon_tab_helper.cc
|
| @@ -4,7 +4,6 @@
|
|
|
| #include "chrome/browser/favicon/favicon_tab_helper.h"
|
|
|
| -#include "chrome/browser/favicon/favicon_download_helper.h"
|
| #include "chrome/browser/favicon/favicon_handler.h"
|
| #include "chrome/browser/favicon/favicon_service_factory.h"
|
| #include "chrome/browser/favicon/favicon_util.h"
|
| @@ -13,7 +12,6 @@
|
| #include "chrome/browser/profiles/profile.h"
|
| #include "chrome/common/chrome_constants.h"
|
| #include "chrome/common/chrome_notification_types.h"
|
| -#include "chrome/common/icon_messages.h"
|
| #include "content/public/browser/favicon_status.h"
|
| #include "content/public/browser/invalidate_type.h"
|
| #include "content/public/browser/navigation_controller.h"
|
| @@ -30,6 +28,7 @@
|
| #include "ui/gfx/image/image_skia_rep.h"
|
|
|
| using content::FaviconStatus;
|
| +using content::FaviconURL;
|
| using content::NavigationController;
|
| using content::NavigationEntry;
|
| using content::WebContents;
|
| @@ -39,7 +38,7 @@ DEFINE_WEB_CONTENTS_USER_DATA_KEY(FaviconTabHelper)
|
| FaviconTabHelper::FaviconTabHelper(WebContents* web_contents)
|
| : content::WebContentsObserver(web_contents),
|
| profile_(Profile::FromBrowserContext(web_contents->GetBrowserContext())) {
|
| - favicon_download_helper_.reset(new FaviconDownloadHelper(web_contents, this));
|
| + web_contents->RegisterFaviconDelegate(this);
|
| favicon_handler_.reset(new FaviconHandler(profile_, this,
|
| FaviconHandler::FAVICON));
|
| if (chrome::kEnableTouchIcon)
|
| @@ -95,7 +94,7 @@ bool FaviconTabHelper::ShouldDisplayFavicon() {
|
| return true;
|
| }
|
|
|
| -void FaviconTabHelper::OnUpdateFaviconURL(
|
| +void FaviconTabHelper::UpdateFaviconURL(
|
| int32 page_id,
|
| const std::vector<FaviconURL>& candidates) {
|
| favicon_handler_->OnUpdateFaviconURL(page_id, candidates);
|
| @@ -134,7 +133,7 @@ NavigationEntry* FaviconTabHelper::GetActiveEntry() {
|
| }
|
|
|
| int FaviconTabHelper::StartDownload(const GURL& url, int image_size) {
|
| - return favicon_download_helper_->DownloadFavicon(url, image_size);
|
| + return web_contents()->DownloadFavicon(url, image_size);
|
| }
|
|
|
| void FaviconTabHelper::NotifyFaviconUpdated() {
|
| @@ -164,7 +163,7 @@ void FaviconTabHelper::DidNavigateMainFrame(
|
| FetchFavicon(details.entry->GetURL());
|
| }
|
|
|
| -void FaviconTabHelper::OnDidDownloadFavicon(
|
| +void FaviconTabHelper::DidDownloadFavicon(
|
| int id,
|
| const GURL& image_url,
|
| bool errored,
|
|
|