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

Unified Diff: components/autofill/core/common/password_form.cc

Issue 1401723002: Show source of affiliated matches in the drop-down password suggestions list. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Test fix Created 5 years, 2 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/autofill/core/common/password_form.cc
diff --git a/components/autofill/core/common/password_form.cc b/components/autofill/core/common/password_form.cc
index 2066ce8b712e3af4e16cbb380f0e9d9c91666fba..0ac9e335b91238dbf9d5b31926eb346621ca473a 100644
--- a/components/autofill/core/common/password_form.cc
+++ b/components/autofill/core/common/password_form.cc
@@ -22,6 +22,8 @@ void PasswordFormToJSON(const PasswordForm& form,
target->SetInteger("scheme", form.scheme);
target->SetString("signon_realm", form.signon_realm);
target->SetBoolean("is_public_suffix_match", form.is_public_suffix_match);
+ target->SetBoolean("is_affiliation_based_match",
+ form.is_affiliation_based_match);
target->SetString("origin", form.origin.possibly_invalid_spec());
target->SetString("action", form.action.possibly_invalid_spec());
target->SetString("submit_element", form.submit_element);
@@ -83,7 +85,7 @@ PasswordForm::PasswordForm()
was_parsed_using_autofill_predictions(false),
is_alive(true),
is_public_suffix_match(false),
- is_affiliated(false) {}
+ is_affiliation_based_match(false) {}
PasswordForm::~PasswordForm() {
CHECK(is_alive);
@@ -123,7 +125,8 @@ bool PasswordForm::operator==(const PasswordForm& form) const {
skip_zero_click == form.skip_zero_click && layout == form.layout &&
was_parsed_using_autofill_predictions ==
form.was_parsed_using_autofill_predictions &&
- is_public_suffix_match == form.is_public_suffix_match;
+ is_public_suffix_match == form.is_public_suffix_match &&
+ is_affiliation_based_match == form.is_affiliation_based_match;
}
bool PasswordForm::operator!=(const PasswordForm& form) const {
« no previous file with comments | « components/autofill/core/common/password_form.h ('k') | components/autofill/core/common/password_form_fill_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698