Enable Automatic Passwords Saving

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 "base/command_line.h"
+
 #include "chrome/browser/password_manager/chrome_password_manager_client.h"
 
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "components/password_manager/core/browser/password_manager_logger.h"
+#include "components/password_manager/core/common/password_manager_switches.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/web_contents.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::BrowserContext;
 using content::WebContents;
 
 namespace {
 
@@ skipping 50 matching lines @@
 TEST_F(ChromePasswordManagerClientTest, LogSavePasswordProgressDetachLogger) {
   ChromePasswordManagerClient* client = GetClient();
 
   client->SetLogger(&logger);
   // After detaching the logger, no text should be passed.
   client->SetLogger(NULL);
   EXPECT_CALL(logger, LogSavePasswordProgress(kTestText)).Times(0);
   client->LogSavePasswordProgress(kTestText);
   EXPECT_FALSE(client->IsLoggingActive());
 }
+
+TEST_F(ChromePasswordManagerClientTest,
+       IsAutomaticPasswordSavingEnabledDefaultBehaviourTest) {
+  ChromePasswordManagerClient* client = GetClient();
+
+  EXPECT_FALSE(client->IsAutomaticPasswordSavingEnabled());

[vabr (Chromium), 2014/04/29 13:04:44]

nit: get rid of the |client| variable and just call
GetClient()->IsAutomaticPasswordSavingEnabled().
That will make the code shorter and easier to read.

The same below.

[rchtara, 2014/04/29 13:51:24]

Done.

+}
+
+TEST_F(ChromePasswordManagerClientTest,
+       IsAutomaticPasswordSavingEnabledWhenFlagIsSetTest) {
+  ChromePasswordManagerClient* client = GetClient();
+
+  CommandLine::ForCurrentProcess()->AppendSwitch(
+      password_manager::switches::kEnableAutomaticPasswordSaving);
+
+  EXPECT_TRUE(client->IsAutomaticPasswordSavingEnabled());
+}