Use upstream libaddressinput in Chrome.

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/macros.h"
 #include "base/strings/string_split.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "components/autofill/core/browser/address_i18n.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/components_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 "third_party/libaddressinput/src/cpp/include/libaddressinput/address_data.h"
+#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_field.h"
+#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_metadata.h"
+#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_ui.h"
+#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_ui_component.h"
+#include "third_party/libaddressinput/src/cpp/include/libaddressinput/localization.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;
 
 DetailInput::Length LengthFromHint(AddressUiComponent::LengthHint hint) {
   if (hint == AddressUiComponent::HINT_SHORT)
     return DetailInput::SHORT;
   DCHECK_EQ(hint, AddressUiComponent::HINT_LONG);
   return DetailInput::LONG;
 }
 
 }  // namespace
 
 void BuildAddressInputs(common::AddressType address_type,
                         const std::string& country_code,
                         DetailInputs* inputs,
                         std::string* language_code) {
+  DCHECK(inputs);
+  DCHECK(language_code);
+
+  ::i18n::addressinput::Localization localization;
+  localization.SetGetter(
+      l10n_util::GetStringUTF8, g_browser_process->GetApplicationLocale());
   std::vector<AddressUiComponent> components(
       ::i18n::addressinput::BuildComponents(
-          country_code, g_browser_process->GetApplicationLocale(),
-          language_code));
+           country_code, localization, language_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 = TypeForField(component.field, address_type);
     DetailInput::Length length = LengthFromHint(component.length_hint);
-    base::string16 placeholder = l10n_util::GetStringUTF16(component.name_id);
+    base::string16 placeholder = base::UTF8ToUTF16(component.name);
     DetailInput input = { length, server_type, placeholder };
     inputs->push_back(input);
   }
 
   ServerFieldType server_type =
       billing ? ADDRESS_BILLING_COUNTRY : ADDRESS_HOME_COUNTRY;
   base::string16 placeholder_text =
       l10n_util::GetStringUTF16(IDS_AUTOFILL_FIELD_LABEL_COUNTRY);
   DetailInput input = { DetailInput::LONG, server_type, placeholder_text };
   inputs->push_back(input);
@@ skipping 15 matching lines @@
   }
 
   return true;
 }
 
 bool AddressHasCompleteAndVerifiedData(const AutofillProfile& profile,
                                        const std::string& app_locale) {
   if (!profile.IsVerified())
     return false;
 
-  if (!i18n::CreateAddressDataFromAutofillProfile(profile, app_locale)->
-          HasAllRequiredFields()) {

[please use gerrit instead, 2014/06/05 22:22:48]

To reduce reviewer burden: make a HasAllRequiredFields() static helper in
third_party/libaddressinput/chromium/has_all_required_fields.{h,cc}.

[please use gerrit instead, 2014/06/09 23:28:16]

Done.

-    return false;
+  scoped_ptr<AddressData> address(
+      i18n::CreateAddressDataFromAutofillProfile(profile, app_locale));
+
+  // TODO: Think about whether it makes sense to do this special check here in
+  // Chromium, instead of just using the regular address validation feature of
+  // libaddressinput.
+  static const AddressField kFields[] = {
+    ::i18n::addressinput::COUNTRY,
+    ::i18n::addressinput::ADMIN_AREA,
+    ::i18n::addressinput::LOCALITY,
+    ::i18n::addressinput::DEPENDENT_LOCALITY,
+    ::i18n::addressinput::SORTING_CODE,
+    ::i18n::addressinput::POSTAL_CODE,
+    ::i18n::addressinput::STREET_ADDRESS,
+    ::i18n::addressinput::RECIPIENT
+  };
+
+  for (size_t i = 0; i < arraysize(kFields); ++i) {
+    AddressField field = kFields[i];
+    if (address->IsFieldEmpty(field) &&
+        ::i18n::addressinput::IsFieldRequired(field, address->region_code)) {
+      return false;
+    }
   }
 
-  const ServerFieldType more_required_fields[] = {
+  static const ServerFieldType more_required_fields[] = {
       NAME_FULL,
       PHONE_HOME_WHOLE_NUMBER
   };
 
   for (size_t i = 0; i < arraysize(more_required_fields); ++i) {
-    if (profile.GetRawInfo(more_required_fields[i]).empty())
+    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_STREET_ADDRESS :
                        ADDRESS_HOME_STREET_ADDRESS;
     case ::i18n::addressinput::RECIPIENT:
       return billing ? NAME_BILLING_FULL : NAME_FULL;
-    case ::i18n::addressinput::ORGANIZATION:
-      return COMPANY_NAME;
   }
   NOTREACHED();
   return UNKNOWN_TYPE;
 }
 
 bool FieldForType(ServerFieldType server_type,
                   ::i18n::addressinput::AddressField* field) {
   switch (server_type) {
     case ADDRESS_BILLING_COUNTRY:
     case ADDRESS_HOME_COUNTRY:
@@ skipping 28 matching lines @@
     case ADDRESS_BILLING_STREET_ADDRESS:
     case ADDRESS_BILLING_LINE1:
     case ADDRESS_BILLING_LINE2:
     case ADDRESS_HOME_STREET_ADDRESS:
     case ADDRESS_HOME_LINE1:
     case ADDRESS_HOME_LINE2:
       if (field)
         *field = ::i18n::addressinput::STREET_ADDRESS;
       return true;
     case COMPANY_NAME:
-      if (field)
-        *field = ::i18n::addressinput::ORGANIZATION;
+      // Ignored.
       return true;

[please use gerrit instead, 2014/06/05 22:22:48]

Investigate: this might have to return false due to no field for this
|server_type|.

[please use gerrit instead, 2014/06/09 23:28:16]

Done.

     case NAME_BILLING_FULL:
     case NAME_FULL:
       if (field)
         *field = ::i18n::addressinput::RECIPIENT;
       return true;
     default:
       return false;
   }
 }
 
 bool CountryIsFullySupported(const std::string& country_code) {
   DCHECK_EQ(2U, country_code.size());
+  ::i18n::addressinput::Localization localization;
+  localization.SetGetter(
+      l10n_util::GetStringUTF8, g_browser_process->GetApplicationLocale());
+  std::string best_address_language;

[please use gerrit instead, 2014/06/05 22:22:47]

To reduce reviewer burden: put this into a function BuildCompnents(country-code,
&best_address_langauge) in anonymous namespace.

[please use gerrit instead, 2014/06/09 23:28:16]

Done.

   std::vector< ::i18n::addressinput::AddressUiComponent> components =
       ::i18n::addressinput::BuildComponents(
-          country_code, g_browser_process->GetApplicationLocale(), NULL);
+           country_code, localization, &best_address_language);
   for (size_t i = 0; i < components.size(); ++i) {
     if (components[i].field == ::i18n::addressinput::DEPENDENT_LOCALITY)
       return false;
   }
   return true;
 }
 
 }  // namespace i18ninput
 }  // namespace autofill