Only report PasswordState in Sync for UMA+non-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 <stdint.h>
 
 #include <string>
 #include <tuple>
 
 #include "base/macros.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/scoped_feature_list.h"
+#include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/sync/profile_sync_test_util.h"
 #include "chrome/common/channel_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/core/browser/credentials_filter.h"
 #include "components/password_manager/core/browser/log_manager.h"
 #include "components/password_manager/core/browser/log_receiver.h"
@@ skipping 318 matching lines @@
       PasswordManagerInternalsServiceFactory::GetForBrowserContext(profile());
   DummyLogReceiver log_receiver;
   EXPECT_EQ(std::string(), log_router->RegisterReceiver(&log_receiver));
 
   // But then navigate to a WebUI, there the logging should not be active.
   NavigateAndCommit(GURL("about:password-manager-internals"));
   EXPECT_FALSE(GetClient()->GetLogManager()->IsLoggingActive());
 
   log_router->UnregisterReceiver(&log_receiver);
 }
+
+TEST_F(ChromePasswordManagerClientTest, ShouldAnnotateNavigationEntry) {
+  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()).WillRepeatedly(Return(true));
+
+  // Connect our bool for testing.
+  bool metrics_enabled = false;
+  ChromeMetricsServiceAccessor::SetMetricsAndCrashReportingForTesting(
+      &metrics_enabled);
+
+  // Metrics disabled w/ non-custom passphrase: do not annotate
+  metrics_enabled = false;
+  EXPECT_CALL(*mock_sync_service, IsUsingSecondaryPassphrase())
+      .WillRepeatedly(Return(false));
+  EXPECT_FALSE(
+      ChromePasswordManagerClient::ShouldAnnotateNavigationEntries(profile()));
+
+  // Metrics enabled w/ non-custom passphrase: do annotate

[vabr (Chromium), 2016/09/02 09:12:20]

Please split the 4 blocks at lines 373, 379, 386 and 393 up into 4 different
test cases.
(See go/unit-test-practices: "DO: Write a separate test for each behavior
defined by a method" vs. "DON'T: Use a single test to validate a method that has
multiple effects"

[Nathan Parker, 2016/09/02 22:42:36]

Done.  And I split apart the new test I added.

+  metrics_enabled = true;
+  EXPECT_CALL(*mock_sync_service, IsUsingSecondaryPassphrase())
+      .WillRepeatedly(Return(false));
+  EXPECT_TRUE(
+      ChromePasswordManagerClient::ShouldAnnotateNavigationEntries(profile()));
+
+  // Metrics enabled w/ custom passphrase: do not annotate
+  metrics_enabled = true;
+  EXPECT_CALL(*mock_sync_service, IsUsingSecondaryPassphrase())
+      .WillRepeatedly(Return(true));
+  EXPECT_FALSE(
+      ChromePasswordManagerClient::ShouldAnnotateNavigationEntries(profile()));
+
+  // Metrics disabled w/ custom passphrase: do not annotate
+  metrics_enabled = false;
+  EXPECT_CALL(*mock_sync_service, IsUsingSecondaryPassphrase())
+      .WillRepeatedly(Return(true));
+  EXPECT_FALSE(
+      ChromePasswordManagerClient::ShouldAnnotateNavigationEntries(profile()));
+
+  // Disconnect our bool for testing.
+  ChromeMetricsServiceAccessor::SetMetricsAndCrashReportingForTesting(nullptr);
+}