Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(227)

Unified Diff: chrome/browser/favicon/favicon_download_helper.h

Issue 11421051: Remove FaviconUtil::DownloadFavicon (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/favicon/favicon_download_helper.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/favicon/favicon_download_helper.h
diff --git a/chrome/browser/favicon/favicon_download_helper.h b/chrome/browser/favicon/favicon_download_helper.h
index 195b3354da5b47dfdf0d6492f62cffd3eac154c5..eb5c372a07dbf7d721cb2c96c95f7d24eb64938a 100644
--- a/chrome/browser/favicon/favicon_download_helper.h
+++ b/chrome/browser/favicon/favicon_download_helper.h
@@ -33,10 +33,10 @@ class FaviconDownloadHelper : public content::WebContentsObserver {
virtual ~FaviconDownloadHelper();
- // Download the favicon at |url|. Returns the unique id of the download
- // request. The id will be passed to
- // FaviconDownloadHelperDelegate::OnDidDownloadFavicon() once the favicon has
- // been retrieved.
+ // Starts the download of the given favicon |url| and returns the unique id of
+ // the download request. When the download is finished, an
+ // IconHostMsg_DidDownloadFavicon IPC message will be sent and passed on to
+ // the delegate via FaviconDownloadHelperDelegate::OnDidDownloadFavicon().
// Note that |image_size| is a hint for images with multiple sizes. The
// downloaded image is not resized to the given image_size. If 0 is passed,
// the first frame of the image is returned.
« no previous file with comments | « no previous file | chrome/browser/favicon/favicon_download_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698