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

Unified Diff: components/password_manager/core/common/experiments.cc

Issue 1396403004: Remove dependency on Finch experiment PasswordLinkInSettings (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Java comment addressed Created 5 years, 2 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/password_manager/core/common/experiments.cc
diff --git a/components/password_manager/core/common/experiments.cc b/components/password_manager/core/common/experiments.cc
index 167aac14aaac179e8b9fcb8ad334096ce3f5c3c2..9582f2ac600079f86c9b406ef03f2d75626541c0 100644
--- a/components/password_manager/core/common/experiments.cc
+++ b/components/password_manager/core/common/experiments.cc
@@ -5,28 +5,10 @@
#include "components/password_manager/core/common/experiments.h"
#include "base/command_line.h"
-#include "base/metrics/field_trial.h"
#include "components/password_manager/core/common/password_manager_switches.h"
namespace password_manager {
-bool ManageAccountLinkExperimentEnabled() {
- std::string group_name =
- base::FieldTrialList::FindFullName("PasswordLinkInSettings");
-
- base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
- if (command_line->HasSwitch(switches::kDisablePasswordLink))
- return false;
-
- if (command_line->HasSwitch(switches::kEnablePasswordLink))
- return true;
-
- // To match Finch enabling the experiment by default, this method returns true
- // unless explicitly told the experiment is disabled. This ensures trybot
- // coverage of the enabled case.
- return group_name != "Disabled";
-}
-
bool ForceSavingExperimentEnabled() {
return base::CommandLine::ForCurrentProcess()->HasSwitch(
password_manager::switches::kEnablePasswordForceSaving);

Powered by Google App Engine
This is Rietveld 408576698