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

Unified Diff: ios/chrome/browser/experimental_flags.mm

Issue 2480013003: Revert of Enable the iPad Tab Switcher by default. (Closed)
Patch Set: Created 4 years, 1 month 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/experimental_flags.mm
diff --git a/ios/chrome/browser/experimental_flags.mm b/ios/chrome/browser/experimental_flags.mm
index 4069bd48c5e8b76dc2ff2d0820c481e99414acaa..32975b428b551f88c4fe9d568ea67d18d89c5f43 100644
--- a/ios/chrome/browser/experimental_flags.mm
+++ b/ios/chrome/browser/experimental_flags.mm
@@ -87,16 +87,18 @@
}
bool IsTabSwitcherEnabled() {
- // Check if the experimental flag is forced off.
+ // Check if the experimental flag is forced on or off.
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
- if (command_line->HasSwitch(switches::kDisableTabSwitcher)) {
+ if (command_line->HasSwitch(switches::kEnableTabSwitcher)) {
+ return true;
+ } else if (command_line->HasSwitch(switches::kDisableTabSwitcher)) {
return false;
}
- // Check if the finch experiment is turned off.
+ // Check if the finch experiment is turned on.
std::string group_name = base::FieldTrialList::FindFullName("IOSTabSwitcher");
- return !base::StartsWith(group_name, "Disabled",
- base::CompareCase::INSENSITIVE_ASCII);
+ return base::StartsWith(group_name, "Enabled",
+ base::CompareCase::INSENSITIVE_ASCII);
}
bool IsReadingListEnabled() {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698