[Autofill] Autofill fails to show suggestions for credit card split across fields.

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 <algorithm>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/format_macros.h"
 #include "base/memory/ref_counted.h"
@@ skipping 195 matching lines @@
         "Expiration Date", "ccmonth", "", "month", &field);
     form->fields.push_back(field);
   } else {
     test::CreateTestFormField("Expiration Date", "ccmonth", "", "text", &field);
     form->fields.push_back(field);
     test::CreateTestFormField("", "ccyear", "", "text", &field);
     form->fields.push_back(field);
   }
 }
 
+// Helper function to create a form with credit card number spits across 4 input
+// fields.
+void CreateTestCreditCardFormWithNumberSplitAcrossFields(FormData* form) {
+  CreateTestCreditCardFormData(form, true, false);
+
+  // Remove existing card number field.
+  form->fields.erase(form->fields.begin() + 1);
+
+  // Add new 4 |card_number_field|s to the |form|.
+  FormFieldData card_number_field;
+  card_number_field.max_length = 4;
+  test::CreateTestFormField(
+      "Card Number", "cardnumber_1", "", "text", &card_number_field);
+  form->fields.insert(form->fields.begin() + 1, card_number_field);
+
+  test::CreateTestFormField("", "cardnumber_2", "", "text", &card_number_field);
+  form->fields.insert(form->fields.begin() + 2, card_number_field);
+
+  test::CreateTestFormField("", "cardnumber_3", "", "text", &card_number_field);
+  form->fields.insert(form->fields.begin() + 3, card_number_field);
+
+  test::CreateTestFormField("", "cardnumber_4", "", "text", &card_number_field);
+  form->fields.insert(form->fields.begin() + 4, card_number_field);
+}
+
 void ExpectFilledField(const char* expected_label,
                        const char* expected_name,
                        const char* expected_value,
                        const char* expected_form_control_type,
                        const FormFieldData& field) {
   SCOPED_TRACE(expected_label);
   EXPECT_EQ(UTF8ToUTF16(expected_label), field.label);
   EXPECT_EQ(UTF8ToUTF16(expected_name), field.name);
   EXPECT_EQ(UTF8ToUTF16(expected_value), field.value);
   EXPECT_EQ(expected_form_control_type, field.form_control_type);
@@ skipping 665 matching lines @@
 // Test that we return only matching credit card profile suggestions when the
 // selected form field has been partially filled out.
 TEST_F(AutofillManagerTest, GetCreditCardSuggestionsMatchCharacter) {
   // Set up our form data.
   FormData form;
   CreateTestCreditCardFormData(&form, true, false);
   std::vector<FormData> forms(1, form);
   FormsSeen(forms);
 
   FormFieldData field;
-  test::CreateTestFormField("Card Number", "cardnumber", "4", "text", &field);
+  test::CreateTestFormField("Card Number", "cardnumber", "78", "text", &field);
   GetAutofillSuggestions(form, field);
 
   // No suggestions provided, so send an empty vector as the results.
   // This triggers the combined message send.
   AutocompleteSuggestionsReturned(std::vector<base::string16>());
 
   // Test that we sent the right values to the external delegate.
   base::string16 expected_values[] = {ASCIIToUTF16("************3456")};
   base::string16 expected_labels[] = {ASCIIToUTF16("04/12")};
   base::string16 expected_icons[] = {ASCIIToUTF16(kVisaCard)};
@@ skipping 2069 matching lines @@
   FormData form;
   test::CreateTestAddressFormData(&form);
   std::vector<FormData> forms(1, form);
   FormsSeen(forms);
   const FormFieldData& field = form.fields[0];
   GetAutofillSuggestions(form, field);  // should call the delegate's OnQuery()
 
   EXPECT_TRUE(external_delegate_->on_query_seen());
 }
 
+// Test to verify suggestions appears for forms having credit card number split
+// across fields.
+TEST_F(AutofillManagerTest, GetCreditCardSuggestionsForNumberSpitAcrossFields) {
+  // Set up our form data with credit card number split across fields.
+  FormData form;
+  CreateTestCreditCardFormWithNumberSplitAcrossFields(&form);
+  std::vector<FormData> forms(1, form);
+  FormsSeen(forms);
+
+  // Fill the credit card details.
+  GUIDPair guid("00000000-0000-0000-0000-000000000004", 0);
+  GUIDPair empty(std::string(), 0);
+  int response_page_id = 0;
+  FormData response_data;
+  FillAutofillFormDataAndSaveResults(kDefaultPageID,
+                                     form,
+                                     *form.fields.begin(),
+                                     PackGUIDs(guid, empty),
+                                     &response_page_id,
+                                     &response_data);
+
+  ASSERT_EQ(7U, response_data.fields.size());
+
+  // Verify whether suggestions for credit card number spilt across 4 fields
+  // getting populated correctly.

[Ilya Sherman, 2014/10/28 21:48:40]

nit: "getting" -> "are"

[Pritam Nikam, 2014/10/29 08:38:16]

Done.

+  const size_t starts_at = 1;
+  const size_t total_number_fields = 4;
+  for (size_t i = starts_at; i < (starts_at + total_number_fields); ++i) {

[Ilya Sherman, 2014/10/28 21:48:40]

Please unroll this loop, so have four repeated sets of checks, rather than
generalizing the structure to a loop.  In tests, the exact number of iterations
is known ahead of time, so there's no need for the expressive power of loops;
and straight-line code makes the test expectations easier to read and
understand.

In fact, for this test, it seems fine to test filling from just one of the
middle fields; there's no need to exhaustively test every possible starting
point for filling.  So, you can just write one set of checks.

[Pritam Nikam, 2014/10/29 08:38:16]

Done.

+    FormFieldData number_field = response_data.fields[i];
+
+    // Enforce 'max-length' constraint on |number_field|.
+    size_t length = (number_field.max_length < number_field.value.size())
+                        ? number_field.max_length
+                        : number_field.value.size();

[Ilya Sherman, 2014/10/28 21:48:40]

You shouldn't need to do this.  Just check that the field starts with the
expected substring, if that's all that the code guarantees.

[Pritam Nikam, 2014/10/29 08:38:16]

Done.

+    base::string16 card_number = number_field.value.substr(0, length);
+    number_field.value = card_number;
+
+    // Get the suggestions for credit card |number_field|.
+    GetAutofillSuggestions(form, number_field);
+
+    // No suggestions provided, so send an empty vector as the results.
+    // This triggers the combined message send.
+    AutocompleteSuggestionsReturned(std::vector<base::string16>());
+
+    base::string16 expected_values[] = {ASCIIToUTF16("************3456")};
+    base::string16 expected_labels[] = {ASCIIToUTF16("04/12")};
+    base::string16 expected_icons[] = {ASCIIToUTF16(kVisaCard)};
+    int expected_unique_ids[] = {autofill_manager_->GetPackedCreditCardID(4)};
+    external_delegate_->CheckSuggestions(kDefaultPageID,
+                                         arraysize(expected_values),
+                                         expected_values,
+                                         expected_labels,
+                                         expected_icons,
+                                         expected_unique_ids);
+  }
+}
+
 }  // namespace autofill