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

Unified Diff: ios/chrome/browser/snapshots/snapshot_cache.mm

Issue 2660143002: Removing iPad Tab Switcher experimental flag. (Closed)
Patch Set: Created 3 years, 11 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: ios/chrome/browser/snapshots/snapshot_cache.mm
diff --git a/ios/chrome/browser/snapshots/snapshot_cache.mm b/ios/chrome/browser/snapshots/snapshot_cache.mm
index d648eba7c3002203e32e2bbe291057bdde45e343..ff319927d62b734ec2f7de700c50564773f423d8 100644
--- a/ios/chrome/browser/snapshots/snapshot_cache.mm
+++ b/ios/chrome/browser/snapshots/snapshot_cache.mm
@@ -405,13 +405,12 @@ void ConvertAndSaveGreyImage(
- (BOOL)inMemoryCacheIsEnabled {
// In-memory cache on iPad is enabled only when the tab switcher is enabled.
- return !IsIPadIdiom() || experimental_flags::IsTabSwitcherEnabled();
+ return !IsIPadIdiom();
rohitrao (ping after 24h) 2017/02/01 18:34:34 It looks like the old implementation always return
jif 2017/02/03 12:55:50 It is! Thank you. Done.
}
- (BOOL)usesLRUCache {
// Always use the LRUCache when the tab switcher is enabled.
- return experimental_flags::IsTabSwitcherEnabled() ||
- experimental_flags::IsLRUSnapshotCacheEnabled();
+ return experimental_flags::IsLRUSnapshotCacheEnabled();
rohitrao (ping after 24h) 2017/02/01 18:34:34 Same here.
jif 2017/02/03 12:55:51 Done.
}
- (void)handleLowMemory {

Powered by Google App Engine
This is Rietveld 408576698