Autofill: Set requirements for number of recognized fields in an autofillable form

components/autofill/core/browser/form_field.h

Index: components/autofill/core/browser/form_field.h
diff --git a/components/autofill/core/browser/form_field.h b/components/autofill/core/browser/form_field.h
index 43714ad60a70c682c237d0bd2ed81d2465a51f78..8f6d33a4d848ed16af48b6fa8797d97e423e7c17 100644
--- a/components/autofill/core/browser/form_field.h
+++ b/components/autofill/core/browser/form_field.h
@@ -29,6 +29,7 @@ class FormField {
   // The association is stored into |map|.  Each field has a derived unique name
   // that is used as the key into the |map|.
   static void ParseFormFields(const std::vector<AutofillField*>& fields,
+                              bool is_unowned_form,

[Evan Stade, 2015/01/21 22:52:50]

nit: |unowned_form| is a slight misnomer. How about |is_in_a_form|?

[Lei Zhang, 2015/01/22 08:07:37]

is_in_form_tag

                               ServerFieldTypeMap* map);
 
  protected:
@@ -88,6 +89,9 @@ class FormField {
   // the type extraction via this method.
   virtual bool ClassifyField(ServerFieldTypeMap* map) const = 0;
 
+  // Returns the number of recognized fields.
+  virtual size_t FieldCount() const = 0;
+
  private:
   FRIEND_TEST_ALL_PREFIXES(FormFieldTest, Match);
 
@@ -115,9 +119,10 @@ class FormField {
   // |fields| is both an input and an output parameter.  Upon exit |fields|
   // holds any remaining unclassified fields for further processing.
   // Classification results of the processed fields are stored in |map|.
-  static void ParseFormFieldsPass(ParseFunction parse,
-                                  std::vector<AutofillField*>* fields,
-                                  ServerFieldTypeMap* map);
+  // Returns the number of recognized fields.
+  static size_t ParseFormFieldsPass(ParseFunction parse,
+                                    std::vector<AutofillField*>* fields,
+                                    ServerFieldTypeMap* map);
 
   // Returns true iff |type| matches |match_type|.
   static bool MatchesFormControlType(const std::string& type, int match_type);