Remove experiment support for PasswordManagerSettingsBehaviourChange

chrome/browser/password_manager/chrome_password_manager_client_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 
 #include <stdint.h>
 
 #include <string>
 
@@ skipping 37 matching lines @@
 using browser_sync::ProfileSyncServiceMock;
 using content::BrowserContext;
 using content::WebContents;
 using sessions::GetPasswordStateFromNavigation;
 using sessions::SerializedNavigationEntry;
 using testing::Return;
 using testing::_;
 
 namespace {
 
-const char kPasswordManagerSettingsBehaviourChangeFieldTrialName[] =
-    "PasswordManagerSettingsBehaviourChange";
-const char kPasswordManagerSettingsBehaviourChangeEnabledGroupName[] =
-    "PasswordManagerSettingsBehaviourChange.Active";
-const char kPasswordManagerSettingsBehaviourChangeDisabledGroupName[] =
-    "PasswordManagerSettingsBehaviourChange.NotActive";
-
 // TODO(vabr): Get rid of the mocked client in the client's own test, see
 // http://crbug.com/474577.
 class MockChromePasswordManagerClient : public ChromePasswordManagerClient {
  public:
   MOCK_CONST_METHOD0(DidLastPageLoadEncounterSSLErrors, bool());
 
   explicit MockChromePasswordManagerClient(content::WebContents* web_contents)
       : ChromePasswordManagerClient(web_contents, nullptr) {
     ON_CALL(*this, DidLastPageLoadEncounterSSLErrors())
         .WillByDefault(testing::Return(false));
@@ skipping 69 matching lines @@
  public:
   ChromePasswordManagerClientTest()
       : field_trial_list_(nullptr), metrics_enabled_(false) {}
   void SetUp() override;
   void TearDown() override;
 
   sync_preferences::TestingPrefServiceSyncable* prefs() {
     return profile()->GetTestingPrefService();
   }
 
-  void EnforcePasswordManagerSettingsBehaviourChangeExperimentGroup(
-      const char* name) {
-    ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial(
-        kPasswordManagerSettingsBehaviourChangeFieldTrialName, name));
-  }
-
   // Caller does not own the returned pointer.
   ProfileSyncServiceMock* SetupBasicMockSync() {
     ProfileSyncServiceMock* mock_sync_service =
         static_cast<ProfileSyncServiceMock*>(
             ProfileSyncServiceFactory::GetInstance()->SetTestingFactoryAndUse(
                 profile(), BuildMockProfileSyncService));
 
     EXPECT_CALL(*mock_sync_service, IsFirstSetupComplete())
         .WillRepeatedly(Return(true));
     EXPECT_CALL(*mock_sync_service, IsSyncActive())
@@ skipping 163 matching lines @@
   // preference.
   ChromePasswordManagerClient* client = GetClient();
   prefs()->SetUserPref(password_manager::prefs::kPasswordManagerSavingEnabled,
                        new base::FundamentalValue(true));
   EXPECT_TRUE(client->IsSavingAndFillingEnabledForCurrentPage());
   prefs()->SetUserPref(password_manager::prefs::kPasswordManagerSavingEnabled,
                        new base::FundamentalValue(false));
   EXPECT_FALSE(client->IsSavingAndFillingEnabledForCurrentPage());
 }
 
-TEST_F(ChromePasswordManagerClientTest,
-       FillingDependsOnManagerEnabledPreferenceAndExperimentEnabled) {
-  // Test that filing of passwords depends on the password manager enabled
-  // preference and is the user participated in behavior change experiment.
-  ChromePasswordManagerClient* client = GetClient();
-  EnforcePasswordManagerSettingsBehaviourChangeExperimentGroup(
-      kPasswordManagerSettingsBehaviourChangeEnabledGroupName);
-  prefs()->SetUserPref(password_manager::prefs::kPasswordManagerSavingEnabled,
-                       new base::FundamentalValue(true));
-  EXPECT_TRUE(client->IsSavingAndFillingEnabledForCurrentPage());
-  EXPECT_TRUE(client->IsFillingEnabledForCurrentPage());
-  prefs()->SetUserPref(password_manager::prefs::kPasswordManagerSavingEnabled,
-                       new base::FundamentalValue(false));
-  EXPECT_FALSE(client->IsSavingAndFillingEnabledForCurrentPage());
-  EXPECT_FALSE(client->IsFillingEnabledForCurrentPage());
-}
-
-TEST_F(ChromePasswordManagerClientTest,
-       FillingDependsOnManagerEnabledPreferenceAndExperimentDisabled) {
-  // Test that filing of passwords depends on the password manager enabled
-  // preference and is the user participated in behavior change experiment.
-  ChromePasswordManagerClient* client = GetClient();
-  EnforcePasswordManagerSettingsBehaviourChangeExperimentGroup(
-      kPasswordManagerSettingsBehaviourChangeDisabledGroupName);
-  prefs()->SetUserPref(password_manager::prefs::kPasswordManagerSavingEnabled,
-                       new base::FundamentalValue(true));
-  EXPECT_TRUE(client->IsFillingEnabledForCurrentPage());
-  prefs()->SetUserPref(password_manager::prefs::kPasswordManagerSavingEnabled,
-                       new base::FundamentalValue(false));
-  EXPECT_TRUE(client->IsFillingEnabledForCurrentPage());
-}
-
 TEST_F(ChromePasswordManagerClientTest, SavingAndFillingEnabledConditionsTest) {
   std::unique_ptr<WebContents> test_web_contents(
       content::WebContentsTester::CreateTestWebContents(
           web_contents()->GetBrowserContext(), nullptr));
   std::unique_ptr<MockChromePasswordManagerClient> client(
       new MockChromePasswordManagerClient(test_web_contents.get()));
   // Functionality disabled if there is SSL errors.
   EXPECT_CALL(*client, DidLastPageLoadEncounterSSLErrors())
       .WillRepeatedly(Return(true));
   EXPECT_FALSE(client->IsSavingAndFillingEnabledForCurrentPage());
@@ skipping 179 matching lines @@
   // In particular, this WebContent should not have the
   // ChromePasswordManagerClient.
   ASSERT_FALSE(
       ChromePasswordManagerClient::FromWebContents(web_contents.get()));
 
   // This call should not crash.
   ChromePasswordManagerClient::BindCredentialManager(
       web_contents->GetMainFrame(),
       password_manager::mojom::CredentialManagerRequest());
 }