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

Unified Diff: components/autofill/content/public/cpp/autofill_types_struct_traits.cc

Issue 2318533002: [Password Generation] Use signatures for form matching (Closed)
Patch Set: Rebase Created 4 years, 3 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/content/public/cpp/autofill_types_struct_traits.cc
diff --git a/components/autofill/content/public/cpp/autofill_types_struct_traits.cc b/components/autofill/content/public/cpp/autofill_types_struct_traits.cc
index 3ea26e928f86cdba22a44a542b8cca81ea332c68..4b020779bdb95636d08d04cbfa6d146195891a31 100644
--- a/components/autofill/content/public/cpp/autofill_types_struct_traits.cc
+++ b/components/autofill/content/public/cpp/autofill_types_struct_traits.cc
@@ -513,10 +513,8 @@ bool StructTraits<mojom::PasswordFormGenerationDataDataView,
PasswordFormGenerationData>::
Read(mojom::PasswordFormGenerationDataDataView data,
PasswordFormGenerationData* out) {
- if (!data.ReadName(&out->name) || !data.ReadAction(&out->action) ||
- !data.ReadGenerationField(&out->generation_field))
- return false;
-
+ out->form_signature = data.form_signature();
+ out->field_signature = data.field_signature();
return true;
}

Powered by Google App Engine
This is Rietveld 408576698