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

Unified Diff: components/favicon/core/favicon_service.h

Issue 2721363002: Extend LargeIconService to fetch missing favicons from a Google server (Closed)
Patch Set: Add fallback_opts=TYPE Created 3 years, 9 months 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 | « components/favicon/core/DEPS ('k') | components/favicon/core/favicon_service_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/favicon/core/favicon_service.h
diff --git a/components/favicon/core/favicon_service.h b/components/favicon/core/favicon_service.h
index 10a8174747ea6890cb385dae6e1d2b2649ff97c1..8ac749f8419265319ec846df441256272f6d6e6b 100644
--- a/components/favicon/core/favicon_service.h
+++ b/components/favicon/core/favicon_service.h
@@ -189,6 +189,20 @@ class FaviconService : public KeyedService {
favicon_base::IconType icon_type,
const gfx::Image& image) = 0;
+ // Same as SetFavicons() with three differences:
+ // 1) It will be a no-op if there is an existing cached favicon for *any* type
+ // for |page_url|.
+ // 2) If |icon_url| is known to the database, |bitmaps| will be ignored (i.e.
+ // the icon won't be overwritten) but the mappings from |page_url| to
+ // |icon_url| will be stored (conditioned to point 1 above).
+ // 3) If |icon_url| is stored, it will be marked as expired.
+ // The callback will receive whether the write actually happened.
+ virtual void SetLastResortFavicons(const GURL& page_url,
+ const GURL& icon_url,
+ favicon_base::IconType icon_type,
+ const gfx::Image& image,
+ base::Callback<void(bool)> callback) = 0;
+
// Avoid repeated requests to download missing favicon.
virtual void UnableToDownloadFavicon(const GURL& icon_url) = 0;
virtual bool WasUnableToDownloadFavicon(const GURL& icon_url) const = 0;
« no previous file with comments | « components/favicon/core/DEPS ('k') | components/favicon/core/favicon_service_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698