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

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

Issue 2133953002: PasswordForm -> FormDigest for GetLogins (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@413020_ssl_valid
Patch Set: Just rebased Created 4 years, 5 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
Index: components/password_manager/core/browser/credential_manager_pending_request_task.cc
diff --git a/components/password_manager/core/browser/credential_manager_pending_request_task.cc b/components/password_manager/core/browser/credential_manager_pending_request_task.cc
index dc995b0ad2a013c860242e2081edc6c6166207f9..e3b7e139d7804705cdf9da4ce7b8321416ff98ba 100644
--- a/components/password_manager/core/browser/credential_manager_pending_request_task.cc
+++ b/components/password_manager/core/browser/credential_manager_pending_request_task.cc
@@ -94,7 +94,8 @@ void CredentialManagerPendingRequestTask::OnGetPasswordStoreResults(
local_results.push_back(form);
form = nullptr;
} else if (affiliated_realms_.count(form->signon_realm) &&
- AffiliatedMatchHelper::IsValidAndroidCredential(*form)) {
+ AffiliatedMatchHelper::IsValidAndroidCredential(
+ PasswordStore::FormDigest(*form))) {
form->is_affiliation_based_match = true;
affiliated_results.push_back(form);
form = nullptr;

Powered by Google App Engine
This is Rietveld 408576698