Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(366)

Unified Diff: components/password_manager/core/browser/password_manager_unittest.cc

Issue 2760883002: [Merge to M-58] [Password Manager] Fix saving for accounts.google.com. (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/password_manager/core/browser/password_manager.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/password_manager/core/browser/password_manager_unittest.cc
diff --git a/components/password_manager/core/browser/password_manager_unittest.cc b/components/password_manager/core/browser/password_manager_unittest.cc
index b0b4e53355c8b8c7568bf077467903e94ee4d76f..894a67cd2c62327c392d04fccce68381d285caf0 100644
--- a/components/password_manager/core/browser/password_manager_unittest.cc
+++ b/components/password_manager/core/browser/password_manager_unittest.cc
@@ -619,7 +619,7 @@ TEST_F(PasswordManagerTest, ReportFormLoginSuccessAndShouldSaveCalled) {
.WillRepeatedly(Return(true));
EXPECT_CALL(client_, GetPrefs()).WillRepeatedly(Return(nullptr));
- manager()->ProvisionallySavePassword(observed_form);
+ manager()->ProvisionallySavePassword(observed_form, nullptr);
// Chrome should recognise the successful login and call
// ReportFormLoginSuccess.
@@ -653,7 +653,7 @@ TEST_F(PasswordManagerTest, SyncCredentialsNotDroppedIfUpToDate) {
EXPECT_CALL(client_, IsSavingAndFillingEnabledForCurrentPage())
.WillRepeatedly(Return(true));
EXPECT_CALL(client_, GetPrefs()).WillRepeatedly(Return(nullptr));
- manager()->ProvisionallySavePassword(form);
+ manager()->ProvisionallySavePassword(form, nullptr);
// Chrome should not remove the sync credential, because it was successfully
// used as stored, and therefore is up to date.
@@ -686,7 +686,7 @@ TEST_F(PasswordManagerTest, SyncCredentialsDroppedWhenObsolete) {
EXPECT_CALL(client_, IsSavingAndFillingEnabledForCurrentPage())
.WillRepeatedly(Return(true));
EXPECT_CALL(client_, GetPrefs()).WillRepeatedly(Return(nullptr));
- manager()->ProvisionallySavePassword(updated_form);
+ manager()->ProvisionallySavePassword(updated_form, nullptr);
client_.FilterAllResultsForSaving();
« no previous file with comments | « components/password_manager/core/browser/password_manager.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698