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

Unified Diff: chrome/browser/bookmarks/enhanced_bookmarks_features.cc

Issue 1202713002: Replace enhanced bookmarks salient images with favicons (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Line wraps in EnhanacedBookmarkItem.java Created 5 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: chrome/browser/bookmarks/enhanced_bookmarks_features.cc
diff --git a/chrome/browser/bookmarks/enhanced_bookmarks_features.cc b/chrome/browser/bookmarks/enhanced_bookmarks_features.cc
index 97f6ccbe90975558f6279603e09e7dd3c0428386..1d80c7cae65aece41442a4af864fd3ffcb3a8991 100644
--- a/chrome/browser/bookmarks/enhanced_bookmarks_features.cc
+++ b/chrome/browser/bookmarks/enhanced_bookmarks_features.cc
@@ -36,22 +36,6 @@ bool GetBookmarksExperimentExtensionID(std::string* extension_id) {
} // namespace
-#if defined(OS_ANDROID)
-bool IsEnhancedBookmarkImageFetchingEnabled(const PrefService* user_prefs) {
- if (IsEnhancedBookmarksEnabled())
- return true;
-
- // Salient images are collected from visited bookmarked pages even if the
- // enhanced bookmark feature is turned off. This is to have some images
- // available so that in the future, when the feature is turned on, the user
- // experience is not a big list of flat colors. However as a precautionary
- // measure it is possible to disable this collection of images from finch.
- std::string disable_fetching = variations::GetVariationParamValue(
- kFieldTrialName, "DisableImagesFetching");
- return disable_fetching.empty();
-}
-#endif // defined(OS_ANDROID)
-
bool IsEnhancedBookmarksEnabled() {
std::string extension_id;
return IsEnhancedBookmarksEnabled(&extension_id);

Powered by Google App Engine
This is Rietveld 408576698