[Password Generation] Use signatures for form matching

components/autofill/core/common/password_form_generation_data.h

Index: components/autofill/core/common/password_form_generation_data.h
diff --git a/components/autofill/core/common/password_form_generation_data.h b/components/autofill/core/common/password_form_generation_data.h
index 38910c904f34b6d612e3ab1eddb2dcb23257f077..4b490a26b8271cac177232c3ac6e28a6b90c00f8 100644
--- a/components/autofill/core/common/password_form_generation_data.h
+++ b/components/autofill/core/common/password_form_generation_data.h
@@ -5,8 +5,11 @@
 #ifndef COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_GENERATION_DATA_H_
 #define COMPONENTS_AUTOFILL_CORE_COMMON_PASSWORD_FORM_GENERATION_DATA_H_
 
+#include <stdint.h>
+
 #include "base/strings/string16.h"
 #include "components/autofill/core/common/form_field_data.h"
+#include "components/autofill/core/common/signatures_util.h"
 #include "url/gurl.h"
 
 namespace autofill {
@@ -14,15 +17,9 @@ namespace autofill {
 // Structure used for sending information from browser to renderer about on
 // which fields password should be generated.
 struct PasswordFormGenerationData {
-  // The name of the form.
-  base::string16 name;
-
-  // The action target of the form; this URL consists of the scheme, host, port
-  // and path; the rest is stripped.
-  GURL action;
+  FormSignature form_signature;

[dcheng, 2016/09/07 19:56:39]

This (or the mojom or both) needs some comments to document what these fields
mean.

[kolos1, 2016/09/08 08:34:27]

Done.

 
-  // Field in which password should be generated.
-  FormFieldData generation_field;
+  FieldSignature field_signature;
 };
 
 }  // namespace autofill