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

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

Issue 2589763002: Enable the iPad Tab Switcher by default. (Closed)
Patch Set: Rebased. Created 4 years 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 b6e3ff20f89ef8f4e010e20a72639190ef7da230..470c755ba4497a25128c9cd01d629b203c1c2c70 100644
--- a/ios/chrome/browser/experimental_flags.mm
+++ b/ios/chrome/browser/experimental_flags.mm
@@ -274,18 +274,16 @@ bool IsStartupCrashEnabled() {
}
bool IsTabSwitcherEnabled() {
- // Check if the experimental flag is forced on or off.
+ // Check if the experimental flag is forced off.
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
- if (command_line->HasSwitch(switches::kEnableTabSwitcher)) {
- return true;
- } else if (command_line->HasSwitch(switches::kDisableTabSwitcher)) {
+ if (command_line->HasSwitch(switches::kDisableTabSwitcher)) {
return false;
}
- // Check if the finch experiment is turned on.
+ // Check if the finch experiment is turned off.
std::string group_name = base::FieldTrialList::FindFullName("IOSTabSwitcher");
- return base::StartsWith(group_name, "Enabled",
- base::CompareCase::INSENSITIVE_ASCII);
+ return !base::StartsWith(group_name, "Disabled",
+ base::CompareCase::INSENSITIVE_ASCII);
}
bool IsViewCopyPasswordsEnabled() {
« 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