Creating a preference for storing a sync password hash.

components/password_manager/core/browser/password_reuse_detector_unittest.cc

Index: components/password_manager/core/browser/password_reuse_detector_unittest.cc
diff --git a/components/password_manager/core/browser/password_reuse_detector_unittest.cc b/components/password_manager/core/browser/password_reuse_detector_unittest.cc
index 09342614b282af93a655f7a89dcde1bf19990042..94d2fa82665967e353f11145145f646ddc411f54 100644
--- a/components/password_manager/core/browser/password_reuse_detector_unittest.cc
+++ b/components/password_manager/core/browser/password_reuse_detector_unittest.cc
@@ -11,6 +11,9 @@
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/common/password_form.h"
 #include "components/password_manager/core/browser/password_manager_test_utils.h"
+#include "components/password_manager/core/common/password_manager_pref_names.h"
+#include "components/prefs/pref_registry_simple.h"
+#include "components/prefs/testing_pref_service.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ -60,7 +63,7 @@ PasswordStoreChangeList GetChangeList(
 }
 
 TEST(PasswordReuseDetectorTest, TypingPasswordOnDifferentSite) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -91,7 +94,7 @@ TEST(PasswordReuseDetectorTest, TypingPasswordOnDifferentSite) {
 }
 
 TEST(PasswordReuseDetectorTest, PSLMatchNoReuseEvent) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -101,7 +104,7 @@ TEST(PasswordReuseDetectorTest, PSLMatchNoReuseEvent) {
 }
 
 TEST(PasswordReuseDetectorTest, NoPSLMatchReuseEvent) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -114,7 +117,7 @@ TEST(PasswordReuseDetectorTest, NoPSLMatchReuseEvent) {
 }
 
 TEST(PasswordReuseDetectorTest, TooShortPasswordNoReuseEvent) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -123,7 +126,7 @@ TEST(PasswordReuseDetectorTest, TooShortPasswordNoReuseEvent) {
 }
 
 TEST(PasswordReuseDetectorTest, PasswordNotInputSuffixNoReuseEvent) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -138,7 +141,7 @@ TEST(PasswordReuseDetectorTest, OnLoginsChanged) {
   for (PasswordStoreChange::Type type :
        {PasswordStoreChange::ADD, PasswordStoreChange::UPDATE,
         PasswordStoreChange::REMOVE}) {
-    PasswordReuseDetector reuse_detector;
+    PasswordReuseDetector reuse_detector(nullptr);
     PasswordStoreChangeList changes =
         GetChangeList(type, GetForms(GetTestDomainsPasswords()));
     reuse_detector.OnLoginsChanged(changes);
@@ -162,7 +165,7 @@ TEST(PasswordReuseDetectorTest, CheckLongestPasswordMatchReturn) {
       {"https://example3.com", "1234567890"},
   };
 
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(domain_passwords));
 
   MockPasswordReuseDetectorConsumer mockConsumer;
@@ -185,7 +188,7 @@ TEST(PasswordReuseDetectorTest, CheckLongestPasswordMatchReturn) {
 }
 
 TEST(PasswordReuseDetectorTest, SyncPasswordNoReuse) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -200,7 +203,7 @@ TEST(PasswordReuseDetectorTest, SyncPasswordNoReuse) {
 }
 
 TEST(PasswordReuseDetectorTest, SyncPasswordReuseFound) {
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -215,7 +218,7 @@ TEST(PasswordReuseDetectorTest, SyncPasswordReuseFound) {
 TEST(PasswordReuseDetectorTest, SavedPasswordsReuseSyncPasswordAvailable) {
   // Check that reuse of saved passwords is detected also if the sync password
   // hash is saved.
-  PasswordReuseDetector reuse_detector;
+  PasswordReuseDetector reuse_detector(nullptr);
   reuse_detector.OnGetPasswordStoreResults(GetForms(GetTestDomainsPasswords()));
   MockPasswordReuseDetectorConsumer mockConsumer;
 
@@ -227,6 +230,16 @@ TEST(PasswordReuseDetectorTest, SavedPasswordsReuseSyncPasswordAvailable) {
                             &mockConsumer);
 }
 
+TEST(PasswordReuseDetectorTest, CheckThatSyncPasswordIsStoredIntoPreferences) {
+  TestingPrefServiceSimple prefs;
+  prefs.registry()->RegisterStringPref(prefs::kSyncPasswordHash, std::string(),
+                                       PrefRegistry::NO_REGISTRATION_FLAGS);
+  ASSERT_FALSE(prefs.HasPrefPath(prefs::kSyncPasswordHash));
+  PasswordReuseDetector reuse_detector(&prefs);
+  reuse_detector.SaveSyncPasswordHash(ASCIIToUTF16("sync_password"));
+  EXPECT_TRUE(prefs.HasPrefPath(prefs::kSyncPasswordHash));
+}
+
 }  // namespace
 
 }  // namespace password_manager