rAc: use libaddressinput to validate international addresses.

chrome/browser/ui/autofill/autofill_dialog_i18n_input.cc

 // Copyright 2014 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 "chrome/browser/ui/autofill/autofill_dialog_i18n_input.h"
 
 #include "base/command_line.h"
+#include "base/strings/string_split.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/common/chrome_switches.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/field_types.h"
 #include "grit/component_strings.h"
+#include "third_party/libaddressinput/chromium/cpp/include/libaddressinput/address_data.h"
 #include "third_party/libaddressinput/chromium/cpp/include/libaddressinput/address_field.h"
 #include "third_party/libaddressinput/chromium/cpp/include/libaddressinput/address_ui.h"
 #include "third_party/libaddressinput/chromium/cpp/include/libaddressinput/address_ui_component.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace autofill {
 namespace i18ninput {
 
 namespace {
 
+using base::UTF16ToUTF8;
+using ::i18n::addressinput::AddressData;
 using ::i18n::addressinput::AddressField;
 using ::i18n::addressinput::AddressUiComponent;
 
-ServerFieldType AddressFieldToServerFieldType(AddressField address_field,
-                                              bool billing) {
-  switch (address_field) {
-    case ::i18n::addressinput::COUNTRY:
-      return billing ? ADDRESS_BILLING_COUNTRY : ADDRESS_HOME_COUNTRY;
-    case ::i18n::addressinput::ADMIN_AREA:
-      return billing ? ADDRESS_BILLING_STATE : ADDRESS_HOME_STATE;
-    case ::i18n::addressinput::LOCALITY:
-      return billing ? ADDRESS_BILLING_CITY : ADDRESS_HOME_CITY;
-    case ::i18n::addressinput::DEPENDENT_LOCALITY:
-      return billing ? ADDRESS_BILLING_DEPENDENT_LOCALITY :
-                       ADDRESS_HOME_DEPENDENT_LOCALITY;
-    case ::i18n::addressinput::POSTAL_CODE:
-      return billing ? ADDRESS_BILLING_ZIP : ADDRESS_HOME_ZIP;
-    case ::i18n::addressinput::SORTING_CODE:
-      return billing ? ADDRESS_BILLING_SORTING_CODE : ADDRESS_HOME_SORTING_CODE;
-    case ::i18n::addressinput::STREET_ADDRESS:
-      return billing ? ADDRESS_BILLING_LINE1 : ADDRESS_HOME_LINE1;
-    case ::i18n::addressinput::RECIPIENT:
-      return billing ? NAME_BILLING_FULL : NAME_FULL;
-    case ::i18n::addressinput::ORGANIZATION:
-      return COMPANY_NAME;
-  }
-  NOTREACHED();
-  return UNKNOWN_TYPE;
-}
-
 DetailInput::Length LengthFromHint(AddressUiComponent::LengthHint hint) {
   if (hint == AddressUiComponent::HINT_SHORT)
     return DetailInput::SHORT;
   DCHECK_EQ(hint, AddressUiComponent::HINT_LONG);
   return DetailInput::LONG;
 }
 
 }  // namespace
 
 bool Enabled() {
   CommandLine* command_line = CommandLine::ForCurrentProcess();
   return command_line->HasSwitch(::switches::kEnableAutofillAddressI18n);
 }
 
 void BuildAddressInputs(common::AddressType address_type,
                         const std::string& country_code,
                         DetailInputs* inputs) {
   std::vector<AddressUiComponent> components(
       ::i18n::addressinput::BuildComponents(country_code));
 
   const bool billing = address_type == common::ADDRESS_TYPE_BILLING;
 
   for (size_t i = 0; i < components.size(); ++i) {
     const AddressUiComponent& component = components[i];
     if (component.field == ::i18n::addressinput::ORGANIZATION) {
       // TODO(dbeam): figure out when we actually need this.
       continue;
     }
 
-    ServerFieldType server_type = AddressFieldToServerFieldType(component.field,
-                                                                billing);
+    ServerFieldType server_type = TypeForField(component.field, address_type);
     DetailInput::Length length = LengthFromHint(component.length_hint);
     base::string16 placeholder = l10n_util::GetStringUTF16(component.name_id);
     DetailInput input = { length, server_type, placeholder };
     inputs->push_back(input);
 
     if (component.field == ::i18n::addressinput::STREET_ADDRESS &&
         component.length_hint == AddressUiComponent::HINT_LONG) {
       // TODO(dbeam): support more than 2 address lines. http://crbug.com/324889
       ServerFieldType server_type =
           billing ? ADDRESS_BILLING_LINE2 : ADDRESS_HOME_LINE2;
@@ skipping 35 matching lines @@
 
   base::string16 country_code = profile.GetRawInfo(ADDRESS_HOME_COUNTRY);
   if (country_code.empty())
     return false;
 
   std::vector<AddressField> required_fields =
       ::i18n::addressinput::GetRequiredFields(base::UTF16ToUTF8(country_code));
 
   for (size_t i = 0; i < required_fields.size(); ++i) {
     ServerFieldType type =
-        AddressFieldToServerFieldType(required_fields[i], false);
+        TypeForField(required_fields[i], common::ADDRESS_TYPE_SHIPPING);
     if (profile.GetRawInfo(type).empty())
       return false;
   }
 
   const ServerFieldType more_required_fields[] = {
       NAME_FULL,
       EMAIL_ADDRESS,
       PHONE_HOME_WHOLE_NUMBER
   };
 
   for (size_t i = 0; i < arraysize(more_required_fields); ++i) {
     if (profile.GetRawInfo(more_required_fields[i]).empty())
       return false;
   }
 
   return true;
 }
 
+ServerFieldType TypeForField(AddressField address_field,
+                             common::AddressType address_type) {
+  bool billing = address_type == common::ADDRESS_TYPE_BILLING;
+  switch (address_field) {
+    case ::i18n::addressinput::COUNTRY:
+      return billing ? ADDRESS_BILLING_COUNTRY : ADDRESS_HOME_COUNTRY;
+    case ::i18n::addressinput::ADMIN_AREA:
+      return billing ? ADDRESS_BILLING_STATE : ADDRESS_HOME_STATE;
+    case ::i18n::addressinput::LOCALITY:
+      return billing ? ADDRESS_BILLING_CITY : ADDRESS_HOME_CITY;
+    case ::i18n::addressinput::DEPENDENT_LOCALITY:
+      return billing ? ADDRESS_BILLING_DEPENDENT_LOCALITY :
+                       ADDRESS_HOME_DEPENDENT_LOCALITY;
+    case ::i18n::addressinput::POSTAL_CODE:
+      return billing ? ADDRESS_BILLING_ZIP : ADDRESS_HOME_ZIP;
+    case ::i18n::addressinput::SORTING_CODE:
+      return billing ? ADDRESS_BILLING_SORTING_CODE : ADDRESS_HOME_SORTING_CODE;
+    case ::i18n::addressinput::STREET_ADDRESS:
+      return billing ? ADDRESS_BILLING_LINE1 : ADDRESS_HOME_LINE1;
+    case ::i18n::addressinput::RECIPIENT:
+      return billing ? NAME_BILLING_FULL : NAME_FULL;
+    case ::i18n::addressinput::ORGANIZATION:
+      return COMPANY_NAME;
+  }
+  NOTREACHED();
+  return UNKNOWN_TYPE;
+}
+
+AddressData CreateAddressData(
+    const base::Callback<base::string16(const AutofillType&)>& get_info) {
+  AddressData address_data;
+  address_data.recipient = UTF16ToUTF8(
+      get_info.Run(AutofillType(NAME_FULL)));
+  address_data.country_code = UTF16ToUTF8(
+      get_info.Run(AutofillType(HTML_TYPE_COUNTRY_CODE, HTML_MODE_NONE)));

[Evan Stade, 2014/01/30 17:25:39]

HTML_MODE_NONE is used for AutofillTypes that don't have an HTML type, not for
"either shipping or billing"

see https://codereview.chromium.org/148273010/

[Dan Beam, 2014/01/30 17:43:50]

Done.

+  address_data.administrative_area = UTF16ToUTF8(
+      get_info.Run(AutofillType(ADDRESS_HOME_STATE)));
+  address_data.locality = UTF16ToUTF8(
+      get_info.Run(AutofillType(ADDRESS_HOME_CITY)));
+  address_data.dependent_locality = UTF16ToUTF8(
+      get_info.Run(AutofillType(ADDRESS_HOME_DEPENDENT_LOCALITY)));
+  address_data.sorting_code = UTF16ToUTF8(
+      get_info.Run(AutofillType(ADDRESS_HOME_SORTING_CODE)));
+  address_data.postal_code = UTF16ToUTF8(
+      get_info.Run(AutofillType(ADDRESS_HOME_ZIP)));
+
+  AutofillType address_lines_type(HTML_TYPE_STREET_ADDRESS, HTML_MODE_NONE);

[Evan Stade, 2014/01/30 17:25:39]

ADDRESS_HOME_STREET_ADDRESS

[Dan Beam, 2014/01/30 17:43:50]

Done.

+  base::SplitString(UTF16ToUTF8(get_info.Run(address_lines_type)),
+                    '\n',
+                    &address_data.address_lines);
+
+  return address_data;
+}
+
 }  // namespace i18ninput
 }  // namespace autofill