Autofill server-side heuristics for 2 fields password form

components/autofill/core/common/autofill_constants.h

Index: components/autofill/core/common/autofill_constants.h
diff --git a/components/autofill/core/common/autofill_constants.h b/components/autofill/core/common/autofill_constants.h
index 9c89bddc809dad5c1430d99275022929c320879b..b725f694d300d72e3fd8ea4d2e3e32ad9fcaa4ec 100644
--- a/components/autofill/core/common/autofill_constants.h
+++ b/components/autofill/core/common/autofill_constants.h
@@ -25,6 +25,10 @@ const size_t kRequiredFieldsForPredictionRoutines = 3;
 // servers.
 const size_t kRequiredFieldsForUpload = 3;
 
+// The minimum number of fields in a form that contains only passwords fields to

[Ilya Sherman, 2016/02/09 22:33:39]

nit: s/passwords/password

[dvadym, 2016/02/10 11:58:28]

Done.

+// upload the form to and request predictions from the Autofill servers.
+const size_t kRequiredFieldsAllPasswordsForUploadAndPrediction = 2;

[Ilya Sherman, 2016/02/09 22:33:39]

nit: I think something like kRequiredFieldsForFormsWithOnlyPasswordFields would
be clearer and slightly shorter.  You could probably find a shorter way to write
"FormsWithOnlyPasswordFields" as well =)

[dvadym, 2016/02/10 11:58:28]

Thanks kRequiredFieldsForFormsWithOnlyPasswordFields is clearer. I didn't come
up with something shorter and clear

+
 // Options bitmask values for AutofillHostMsg_ShowPasswordSuggestions IPC
 enum ShowPasswordSuggestionsOptions {
   SHOW_ALL = 1 << 0 /* show all credentials, not just ones matching username */,