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

Unified Diff: chrome/browser/ui/webui/favicon_source.h

Issue 2535463002: [Favicon] Stop caching the default favicon (Closed)
Patch Set: restructure the code Created 4 years 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
Index: chrome/browser/ui/webui/favicon_source.h
diff --git a/chrome/browser/ui/webui/favicon_source.h b/chrome/browser/ui/webui/favicon_source.h
index 0f47e0e4437800e370bfd7e76b5c3c6f4b45c6cf..7e43e00adec656cdadc744b1809dcb0575a5ab0c 100644
--- a/chrome/browser/ui/webui/favicon_source.h
+++ b/chrome/browser/ui/webui/favicon_source.h
@@ -105,14 +105,6 @@ class FaviconSource : public content::URLDataSource {
Profile* profile_;
private:
- // Defines the allowed pixel sizes for requested favicons.
- enum IconSize {
- SIZE_16,
- SIZE_32,
- SIZE_64,
- NUM_SIZES
- };
-
// Called when favicon data is available from the history backend.
void OnFaviconDataAvailable(
const IconRequest& request,
@@ -127,10 +119,6 @@ class FaviconSource : public content::URLDataSource {
base::CancelableTaskTracker cancelable_task_tracker_;
- // Raw PNG representations of favicons of each size to show when the favicon
- // database doesn't have a favicon for a webpage. Indexed by IconSize values.
- scoped_refptr<base::RefCountedMemory> default_favicons_[NUM_SIZES];
-
// The favicon_base::IconTypes of icon that this FaviconSource handles.
int icon_types_;

Powered by Google App Engine
This is Rietveld 408576698