[Autofill] Disambiguate similar types before uploading.

components/autofill/core/browser/autofill_manager_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
@@ skipping 3165 matching lines @@
   const char guid[] = "00000000-0000-0000-0000-000000000001";
   int response_page_id = 0;
   FormData response_data;
   FillAutofillFormDataAndSaveResults(kDefaultPageID, form, form.fields[0],
                                      MakeFrontendID(std::string(), guid),
                                      &response_page_id, &response_data);
   ExpectFilledAddressFormElvis(response_page_id, response_data, kDefaultPageID,
                                false);
 
   personal_data_.ClearAutofillProfiles();
+  // The default credit card is a Elvis card. It must be removed because name
+  // fields would be detected. However at least one profile or card is needed to
+  // start the upload process, which is why this other card is created.
+  personal_data_.ClearCreditCards();
+  personal_data_.CreateTestCreditCardsYearAndMonth("2012", "04");
   ASSERT_EQ(0u, personal_data_.GetProfiles().size());
 
   // Simulate form submission. The first submission should not count the data
   // towards possible types. Therefore we expect all UNKNOWN_TYPE entries.
   ServerFieldTypeSet type_set;
   type_set.insert(UNKNOWN_TYPE);
   std::vector<ServerFieldTypeSet> unknown_types(expected_types.size(),
                                                 type_set);
   autofill_manager_->set_expected_submitted_field_types(unknown_types);
   FormSubmitted(response_data);
@@ skipping 137 matching lines @@
   test::SetProfileInfo(&profile, "Charles", "", "Baudelaire",
                        "lesfleursdumal@gmail.com", "", "108 Rue Saint-Lazare",
                        "Apt. 10", "Paris", "Ile de France", "75008", "FR",
                        "+33 2 49 19 70 70");
   profile.set_guid("00000000-0000-0000-0000-000000000001");
   profiles.push_back(profile);
 
   // Set up the test credit cards.
   std::vector<CreditCard> credit_cards;
   CreditCard credit_card;
-  test::SetCreditCardInfo(&credit_card, "Elvis Presley", "4234-5678-9012-3456",
-                          "04", "2012");
+  test::SetCreditCardInfo(&credit_card, "John Doe", "4234-5678-9012-3456", "04",
+                          "2012");
   credit_card.set_guid("00000000-0000-0000-0000-000000000003");
   credit_cards.push_back(credit_card);
 
   typedef struct {
     std::string input_value;     // The value to input in the field.
     ServerFieldType field_type;  // The expected field type to be determined.
   } TestCase;
 
   TestCase test_cases[] = {
       // Profile fields matches.
@@ skipping 28 matching lines @@
       {"FR", ADDRESS_HOME_COUNTRY},
       {"France", ADDRESS_HOME_COUNTRY},
       {"33249197070", PHONE_HOME_WHOLE_NUMBER},
       {"+33 2 49 19 70 70", PHONE_HOME_WHOLE_NUMBER},
       {"2 49 19 70 70", PHONE_HOME_CITY_AND_NUMBER},
       {"249197070", PHONE_HOME_CITY_AND_NUMBER},
       {"33", PHONE_HOME_COUNTRY_CODE},
       {"2", PHONE_HOME_CITY_CODE},
 
       // Credit card fields matches.
-      {"Elvis Presley", CREDIT_CARD_NAME_FULL},
+      {"John Doe", CREDIT_CARD_NAME_FULL},
+      {"John", CREDIT_CARD_NAME_FIRST},
+      {"Doe", CREDIT_CARD_NAME_LAST},
       {"4234-5678-9012-3456", CREDIT_CARD_NUMBER},
       {"04", CREDIT_CARD_EXP_MONTH},
       {"April", CREDIT_CARD_EXP_MONTH},
       {"2012", CREDIT_CARD_EXP_4_DIGIT_YEAR},
       {"12", CREDIT_CARD_EXP_2_DIGIT_YEAR},
       {"04/2012", CREDIT_CARD_EXP_DATE_4_DIGIT_YEAR},
 
       // Make sure whitespaces and invalid characters are handled properly.
       {"", EMPTY_TYPE},
       {" ", EMPTY_TYPE},
@@ skipping 23 matching lines @@
       {"56", UNKNOWN_TYPE},
       {"901", UNKNOWN_TYPE}};
 
   for (TestCase test_case : test_cases) {
     FormData form;
     form.name = ASCIIToUTF16("MyForm");
     form.origin = GURL("http://myform.com/form.html");
     form.action = GURL("http://myform.com/submit.html");
 
     FormFieldData field;
-    ServerFieldTypeSet types;
     test::CreateTestFormField("", "1", "", "text", &field);
     field.value = ASCIIToUTF16(test_case.input_value);
-    types.clear();
-    types.insert(test_case.field_type);
     form.fields.push_back(field);
 
     FormStructure form_structure(form);
 
     AutofillManager::DeterminePossibleFieldTypesForUpload(
         profiles, credit_cards, "en-us", &form_structure);
 
     ASSERT_EQ(1U, form_structure.field_count());
     ServerFieldTypeSet possible_types =
         form_structure.field(0)->possible_types();
@@ skipping 47 matching lines @@
   // Once the form is cached, fill the values.
   EXPECT_EQ(form.fields.size(), expected_values.size());
   for (size_t i = 0; i < expected_values.size(); i++) {
     form.fields[i].value = expected_values[i];
   }
 
   autofill_manager_->set_expected_submitted_field_types(expected_types);
   FormSubmitted(form);
 }
 
+// Tests that DisambiguateUploadTypes makes the correct choices.
+TEST_F(AutofillManagerTest, DisambiguateUploadTypes) {
+  // Set up the test profile.
+  std::vector<AutofillProfile> profiles;
+  AutofillProfile profile;
+  test::SetProfileInfo(&profile, "Elvis", "Aaron", "Presley",
+                       "theking@gmail.com", "RCA", "3734 Elvis Presley Blvd.",
+                       "", "Memphis", "Tennessee", "38116", "US",
+                       "(234) 567-8901");
+  profile.set_guid("00000000-0000-0000-0000-000000000001");
+  profiles.push_back(profile);
+
+  // Set up the test credit card.
+  std::vector<CreditCard> credit_cards;
+  CreditCard credit_card;
+  test::SetCreditCardInfo(&credit_card, "Elvis Presley", "4234-5678-9012-3456",
+                          "04", "2012");
+  credit_card.set_guid("00000000-0000-0000-0000-000000000003");
+  credit_cards.push_back(credit_card);
+
+  typedef struct {
+    std::string input_value;
+    ServerFieldType predicted_type;
+    bool expect_disambiguation;
+    ServerFieldType expected_upload_type;
+  } TestFieldData;
+
+  std::vector<TestFieldData> test_cases[] = {
+      // Address disambiguation.
+      // An ambiguous address line followed by a field predicted as a line 2 and
+      // that is empty should be disambiguated as an ADDRESS_HOME_LINE1.
+      {

[vabr (Chromium), 2016/03/22 17:08:10]

This is an initializer list, IIUC, and those are currently banned in
http://chromium-cpp.appspot.com/.

Could you make this a TestFieldData test_cases[][] or TestFieldData * const
test_cases[]?

[sebsg, 2016/03/22 18:30:26]

I initialized the vector manually. I think it was the simplest solution. The
others were not as straightforward as they looked unfortunately. I really hope
they allow initializer lists soon.

Thanks for spotting this.

+          {"3734 Elvis Presley Blvd.", ADDRESS_HOME_LINE1, true,
+           ADDRESS_HOME_LINE1},
+          {"", ADDRESS_HOME_LINE2, true, EMPTY_TYPE},
+      },
+      // An ambiguous address line followed by a field predicted as a line 2 but
+      // filled with another know profile value should be disambiguated as an
+      // ADDRESS_HOME_STREET_ADDRESS.
+      {
+          {"3734 Elvis Presley Blvd.", ADDRESS_HOME_STREET_ADDRESS, true,
+           ADDRESS_HOME_STREET_ADDRESS},
+          {"38116", ADDRESS_HOME_LINE2, true, ADDRESS_HOME_ZIP},
+      },
+      // An ambiguous address line followed by an empty field predicted as
+      // something other than a line 2 should be disambiguated as an
+      // ADDRESS_HOME_STREET_ADDRESS.
+      {
+          {"3734 Elvis Presley Blvd.", ADDRESS_HOME_STREET_ADDRESS, true,
+           ADDRESS_HOME_STREET_ADDRESS},
+          {"", ADDRESS_HOME_ZIP, true, EMPTY_TYPE},
+      },
+      // An ambiguous address line followed by no other field should be
+      // disambiguated as an ADDRESS_HOME_STREET_ADDRESS.
+      {
+          {"3734 Elvis Presley Blvd.", ADDRESS_HOME_STREET_ADDRESS, true,
+           ADDRESS_HOME_STREET_ADDRESS},
+      },
+
+      // Phone number disambiguation.
+      // A field with possible types PHONE_HOME_CITY_AND_NUMBER and
+      // PHONE_HOME_WHOLE_NUMBER should be disambiguated as
+      // PHONE_HOME_CITY_AND_NUMBER
+      {
+          {"2345678901", PHONE_HOME_WHOLE_NUMBER, true,
+           PHONE_HOME_CITY_AND_NUMBER},
+      },
+
+      // Name disambiguation.
+      // An ambiguous name field that has no next field and that is preceded by
+      // a non credit card field should be disambiguated as a non credit card
+      // name.
+      {
+          {"Memphis", ADDRESS_HOME_CITY, true, ADDRESS_HOME_CITY},
+          {"Elvis", CREDIT_CARD_NAME_FIRST, true, NAME_FIRST},
+          {"Presley", CREDIT_CARD_NAME_LAST, true, NAME_LAST},
+      },
+      // An ambiguous name field that has no next field and that is preceded by
+      // a credit card field should be disambiguated as a credit card name.
+      {
+          {"4234-5678-9012-3456", CREDIT_CARD_NUMBER, true, CREDIT_CARD_NUMBER},
+          {"Elvis", NAME_FIRST, true, CREDIT_CARD_NAME_FIRST},
+          {"Presley", NAME_LAST, true, CREDIT_CARD_NAME_LAST},
+      },
+      // An ambiguous name field that has no previous field and that is
+      // followed by a non credit card field should be disambiguated as a non
+      // credit card name.
+      {
+          {"Elvis", CREDIT_CARD_NAME_FIRST, true, NAME_FIRST},
+          {"Presley", CREDIT_CARD_NAME_LAST, true, NAME_LAST},
+          {"Memphis", ADDRESS_HOME_CITY, true, ADDRESS_HOME_CITY},
+      },
+      // An ambiguous name field that has no previous field and that is followed
+      // by a credit card field should be disambiguated as a credit card name.
+      {
+          {"Elvis", NAME_FIRST, true, CREDIT_CARD_NAME_FIRST},
+          {"Presley", NAME_LAST, true, CREDIT_CARD_NAME_LAST},
+          {"4234-5678-9012-3456", CREDIT_CARD_NUMBER, true, CREDIT_CARD_NUMBER},
+      },
+      // An ambiguous name field that is preceded and followed by non credit
+      // card fields should be disambiguated as a non credit card name.
+      {
+          {"Memphis", ADDRESS_HOME_CITY, true, ADDRESS_HOME_CITY},
+          {"Elvis", CREDIT_CARD_NAME_FIRST, true, NAME_FIRST},
+          {"Presley", CREDIT_CARD_NAME_LAST, true, NAME_LAST},
+          {"Tennessee", ADDRESS_HOME_STATE, true, ADDRESS_HOME_STATE},
+      },
+      // An ambiguous name field that is preceded and followed by credit card
+      // fields should be disambiguated as a credit card name.
+      {
+          {"4234-5678-9012-3456", CREDIT_CARD_NUMBER, true, CREDIT_CARD_NUMBER},
+          {"Elvis", NAME_FIRST, true, CREDIT_CARD_NAME_FIRST},
+          {"Presley", NAME_LAST, true, CREDIT_CARD_NAME_LAST},
+          {"2012", CREDIT_CARD_EXP_4_DIGIT_YEAR, true,
+           CREDIT_CARD_EXP_4_DIGIT_YEAR},
+      },
+      // An ambiguous name field that is preceded by a non credit card field and
+      // followed by a credit card field should not be disambiguated.
+      {
+          {"Memphis", ADDRESS_HOME_CITY, true, ADDRESS_HOME_CITY},
+          {"Elvis", NAME_FIRST, false, CREDIT_CARD_NAME_FIRST},
+          {"Presley", NAME_LAST, false, CREDIT_CARD_NAME_LAST},
+          {"2012", CREDIT_CARD_EXP_4_DIGIT_YEAR, true,
+           CREDIT_CARD_EXP_4_DIGIT_YEAR},
+      },
+      // An ambiguous name field that is preceded by a credit card field and
+      // followed by a non credit card field should not be disambiguated.
+      {
+          {"2012", CREDIT_CARD_EXP_4_DIGIT_YEAR, true,
+           CREDIT_CARD_EXP_4_DIGIT_YEAR},
+          {"Elvis", NAME_FIRST, false, CREDIT_CARD_NAME_FIRST},
+          {"Presley", NAME_LAST, false, CREDIT_CARD_NAME_LAST},
+          {"Memphis", ADDRESS_HOME_CITY, true, ADDRESS_HOME_CITY},
+      },
+  };
+
+  for (std::vector<TestFieldData> test_fields : test_cases) {

[vabr (Chromium), 2016/03/22 17:08:10]

nit: Please do not copy into test_fields, use a const reference instead. Feel
free to name the type const auto&.

[sebsg, 2016/03/22 18:30:26]

Done.

+    FormData form;
+    form.name = ASCIIToUTF16("MyForm");
+    form.origin = GURL("http://myform.com/form.html");
+    form.action = GURL("http://myform.com/submit.html");
+
+    // Create the form fields specified in the test case.
+    FormFieldData field;
+    for (size_t i = 0; i < test_fields.size(); ++i) {

[vabr (Chromium), 2016/03/22 17:08:10]

Let's use a range-based loop here, it's less clutter.
(I see why that's not possible for the other two loops below.)

[sebsg, 2016/03/22 18:30:26]

Done.

+      test::CreateTestFormField("", "1", "", "text", &field);
+      field.value = ASCIIToUTF16(test_fields[i].input_value);
+      form.fields.push_back(field);
+    }
+
+    // Assign the specified predicted type for each field in the test case.
+    FormStructure form_structure(form);
+    for (size_t i = 0; i < test_fields.size(); ++i) {
+      form_structure.field(i)->set_server_type(test_fields[i].predicted_type);
+    }
+
+    AutofillManager::DeterminePossibleFieldTypesForUpload(
+        profiles, credit_cards, "en-us", &form_structure);
+    ASSERT_EQ(test_fields.size(), form_structure.field_count());
+
+    // Make sure the disambiguation method selects the expected upload type.
+    ServerFieldTypeSet possible_types;
+    for (size_t i = 0; i < test_fields.size(); ++i) {
+      possible_types = form_structure.field(i)->possible_types();
+      if (test_fields[i].expect_disambiguation) {
+        EXPECT_EQ(1U, possible_types.size());
+        EXPECT_NE(possible_types.end(),
+                  possible_types.find(test_fields[i].expected_upload_type));
+      } else {
+        EXPECT_EQ(2U, possible_types.size());
+      }
+    }
+  }
+}
+
 TEST_F(AutofillManagerTest, RemoveProfile) {
   // Add and remove an Autofill profile.
   AutofillProfile* profile = new AutofillProfile;
   const char guid[] = "00000000-0000-0000-0000-000000000102";
   profile->set_guid(guid);
   autofill_manager_->AddProfile(profile);
 
   int id = MakeFrontendID(std::string(), guid);
 
   autofill_manager_->RemoveAutofillProfileOrCreditCard(id);
@@ skipping 1089 matching lines @@
   FormsSeen(mixed_forms);
 
   // Suggestions should always be displayed.
   for (const FormFieldData& field : mixed_form.fields) {
     GetAutofillSuggestions(mixed_form, field);
     EXPECT_TRUE(external_delegate_->on_suggestions_returned_seen());
   }
 }
 
 }  // namespace autofill