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

components/autofill/core/browser/autofill_field.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 "components/autofill/core/browser/autofill_field.h"
 
 #include "base/logging.h"
 #include "base/sha1.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
@@ skipping 256 matching lines @@
     }
   }
 
   // For American Express, also try filling as "AmEx".
   if (value == l10n_util::GetStringUTF16(IDS_AUTOFILL_CC_AMEX))
     return FillCreditCardTypeSelectControl(ASCIIToUTF16("AmEx"), field);
 
   return false;
 }
 
-// Set |field_data|'s value to |number|, or possibly an appropriate substring of
-// |number|.  The |field| specifies the type of the phone and whether this is a
-// phone prefix or suffix.
-void FillPhoneNumberField(const AutofillField& field,
-                          const base::string16& number,
-                          FormFieldData* field_data) {
-  field_data->value =
-      AutofillField::GetPhoneNumberValue(field, number, *field_data);
-}
-
-// Set |field_data|'s value to |number|, or possibly an appropriate substring
-// of |number| for cases where credit card number splits across multiple HTML
-// form input fields.
-// The |field| specifies the |credit_card_number_offset_| to the substring
-// within credit card number.
-void FillCreditCardNumberField(const AutofillField& field,
-                               const base::string16& number,
-                               FormFieldData* field_data) {
-  base::string16 value = number;
-
-  // |field|'s max_length truncates credit card number to fit within.
-  if (field.credit_card_number_offset() < value.length())
-    value = value.substr(field.credit_card_number_offset());
-
-  field_data->value = value;
-}
-
 // Fills in the select control |field| with |value|.  If an exact match is not
 // found, falls back to alternate filling strategies based on the |type|.
 bool FillSelectControl(const AutofillType& type,
                        const base::string16& value,
                        const std::string& app_locale,
                        FormFieldData* field) {
   DCHECK_EQ("select-one", field->form_control_type);
 
   // Guard against corrupted values passed over IPC.
   if (field->option_values.size() != field->option_contents.size())
@@ skipping 161 matching lines @@
 
 // static
 bool AutofillField::FillFormField(const AutofillField& field,
                                   const base::string16& value,
                                   const std::string& address_language_code,
                                   const std::string& app_locale,
                                   FormFieldData* field_data) {
   AutofillType type = field.Type();
 
   if (type.GetStorableType() == PHONE_HOME_NUMBER) {
-    FillPhoneNumberField(field, value, field_data);
+    field_data->value = AutofillField::GetPhoneNumberValue(field, value);
     return true;
   } else if (field_data->form_control_type == "select-one") {
     return FillSelectControl(type, value, app_locale, field_data);
   } else if (field_data->form_control_type == "month") {
     return FillMonthControl(value, field_data);
   } else if (type.GetStorableType() == ADDRESS_HOME_STREET_ADDRESS) {
     FillStreetAddress(value, address_language_code, field_data);
     return true;
   } else if (type.GetStorableType() == CREDIT_CARD_NUMBER) {
-    FillCreditCardNumberField(field, value, field_data);
+    field_data->value = AutofillField::GetCreditCardNumberValue(field, value);
     return true;
   }
 
   field_data->value = value;
   return true;
 }
 
 base::string16 AutofillField::GetPhoneNumberValue(
     const AutofillField& field,
-    const base::string16& number,
-    const FormFieldData& field_data) {
+    const base::string16& number) {
   // Check to see if the size field matches the "prefix" or "suffix" size.
   // If so, return the appropriate substring.
   if (number.length() !=
           PhoneNumber::kPrefixLength + PhoneNumber::kSuffixLength) {
     return number;
   }
 
   if (field.phone_part() == AutofillField::PHONE_PREFIX ||
-      field_data.max_length == PhoneNumber::kPrefixLength) {
+      field.max_length == PhoneNumber::kPrefixLength) {
     return
         number.substr(PhoneNumber::kPrefixOffset, PhoneNumber::kPrefixLength);
   }
 
   if (field.phone_part() == AutofillField::PHONE_SUFFIX ||
-      field_data.max_length == PhoneNumber::kSuffixLength) {
+      field.max_length == PhoneNumber::kSuffixLength) {
     return
         number.substr(PhoneNumber::kSuffixOffset, PhoneNumber::kSuffixLength);
   }
 
   return number;
 }
 
+// The |field| specifies the |credit_card_number_offset()| to the substring
+// within credit card number.
+base::string16 AutofillField::GetCreditCardNumberValue(
+    const AutofillField& field,
+    const base::string16& number) {
+  base::string16 value = number;
+
+  // |field|'s max_length truncates credit card number to fit within.
+  if (field.credit_card_number_offset() < value.length())
+    value = value.substr(field.credit_card_number_offset());
+
+  return value;
+}
+
 }  // namespace autofill