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

Unified Diff: components/favicon/core/large_icon_service.cc

Issue 2856873002: [Thumbnails DB] Allow setting last_requested time when accessing favicons. (Closed)
Patch Set: Peter's comments Created 3 years, 6 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
Index: components/favicon/core/large_icon_service.cc
diff --git a/components/favicon/core/large_icon_service.cc b/components/favicon/core/large_icon_service.cc
index d5dc2d3de36bc954f7d6d9aa59aec210dd395920..61c341ace443c9ff732b52d3b1fcad05997371f4 100644
--- a/components/favicon/core/large_icon_service.cc
+++ b/components/favicon/core/large_icon_service.cc
@@ -297,9 +297,9 @@ void OnFetchIconFromGoogleServerComplete(
// something else could've been written). By marking the icons initially
// expired (out-of-date), they will be refetched when we visit the original
// page any time in the future.
- favicon_service->SetLastResortFavicons(page_url, GURL(original_icon_url),
- favicon_base::IconType::TOUCH_ICON,
- image, callback);
+ favicon_service->SetOnDemandFavicons(page_url, GURL(original_icon_url),
+ favicon_base::IconType::TOUCH_ICON,
+ image, callback);
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698