[Password Manager] Add UMA stats for custom passphrase users.

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 "base/command_line.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
+#include "chrome/browser/sync/profile_sync_service_factory.h"
+#include "chrome/browser/sync/profile_sync_service_mock.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/autofill/content/common/autofill_messages.h"
 #include "components/password_manager/content/browser/password_manager_internals_service_factory.h"
 #include "components/password_manager/content/common/credential_manager_messages.h"
 #include "components/password_manager/content/common/credential_manager_types.h"
 #include "components/password_manager/core/browser/log_receiver.h"
 #include "components/password_manager/core/browser/password_manager_internals_service.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 "content/public/test/mock_render_process_host.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using content::BrowserContext;
 using content::WebContents;
+using testing::Return;
 
 namespace {
 
 const char kTestText[] = "abcd1234";
 
 class MockLogReceiver : public password_manager::LogReceiver {
  public:
   MOCK_METHOD1(LogSavePasswordProgress, void(const std::string&));
 };
 
@@ skipping 285 matching lines @@
       GURL("https://accounts.google.com/ServiceLogin?continue="
            "https://mail.google.com&rart=234"));
   EXPECT_TRUE(client->IsPasswordManagerEnabledForCurrentPage());
 
   // Reauth pages are only on accounts.google.com
   NavigateAndCommit(
       GURL("https://other.site.com/ServiceLogin?continue="
            "https://passwords.google.com&rart=234"));
   EXPECT_TRUE(client->IsPasswordManagerEnabledForCurrentPage());
 }
+
+TEST_F(ChromePasswordManagerClientTest, IsPasswordSyncEnabled) {
+  ChromePasswordManagerClient* client = GetClient();
+
+  ProfileSyncServiceMock* mock_sync_service =
+      static_cast<ProfileSyncServiceMock*>(
+          ProfileSyncServiceFactory::GetInstance()->SetTestingFactoryAndUse(
+              profile(), ProfileSyncServiceMock::BuildMockProfileSyncService));
+
+  syncer::ModelTypeSet active_types;
+  active_types.Put(syncer::PASSWORDS);
+  EXPECT_CALL(*mock_sync_service, HasSyncSetupCompleted())
+      .WillRepeatedly(Return(true));
+  EXPECT_CALL(*mock_sync_service, sync_initialized())
+      .WillRepeatedly(Return(true));
+  EXPECT_CALL(*mock_sync_service, GetActiveDataTypes())
+      .WillRepeatedly(Return(active_types));
+  EXPECT_CALL(*mock_sync_service, IsUsingSecondaryPassphrase())
+      .WillRepeatedly(Return(false));
+
+  // Passwords are syncing and custom passphrase isn't used.
+  EXPECT_FALSE(client->IsPasswordSyncEnabled(
+      password_manager::ONLY_CUSTOM_PASSPHRASE_USERS));
+  EXPECT_TRUE(client->IsPasswordSyncEnabled(
+      password_manager::EXCLUDE_CUSTOM_PASSPHRASE_USERS));
+
+  // Again, using a custom passphrase.
+  EXPECT_CALL(*mock_sync_service, IsUsingSecondaryPassphrase())
+      .WillRepeatedly(Return(true));
+
+  EXPECT_TRUE(client->IsPasswordSyncEnabled(
+      password_manager::ONLY_CUSTOM_PASSPHRASE_USERS));
+  EXPECT_FALSE(client->IsPasswordSyncEnabled(
+      password_manager::EXCLUDE_CUSTOM_PASSPHRASE_USERS));
+
+  // Always return false if we aren't syncing passwords.
+  active_types.Remove(syncer::PASSWORDS);
+  active_types.Put(syncer::BOOKMARKS);
+  EXPECT_CALL(*mock_sync_service, GetActiveDataTypes())
+      .WillRepeatedly(Return(active_types));
+
+  EXPECT_FALSE(client->IsPasswordSyncEnabled(

[vabr (Chromium), 2014/10/13 09:24:18]

Should you also mock the two possible values of IsUsingSecondaryPassphrase()?

[Garrett Casto, 2014/10/13 20:57:56]

Sure, done.

+      password_manager::ONLY_CUSTOM_PASSPHRASE_USERS));
+  EXPECT_FALSE(client->IsPasswordSyncEnabled(
+      password_manager::EXCLUDE_CUSTOM_PASSPHRASE_USERS));
+}