[rac] Parse postal code formats and required fields in libaddressinput.

third_party/libaddressinput/chromium/cpp/src/address_ui.cc

 // Copyright (C) 2013 Google Inc.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 // http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
 #include <libaddressinput/address_ui.h>
 
 #include <libaddressinput/address_field.h>
 #include <libaddressinput/address_ui_component.h>
 #include <libaddressinput/localization.h>
 
 #include <string>
 #include <vector>
 
-#include "address_field_util.h"
 #include "grit.h"
 #include "messages.h"
 #include "region_data_constants.h"
 #include "rule.h"
 
 namespace i18n {
 namespace addressinput {
 
 namespace {
 
@@ skipping 35 matching lines @@
       return IDS_LIBADDRESSINPUT_I18N_ADDRESS_LINE1_LABEL;
     case ORGANIZATION:
       return IDS_LIBADDRESSINPUT_I18N_ORGANIZATION_LABEL;
     case RECIPIENT:
       return IDS_LIBADDRESSINPUT_I18N_RECIPIENT_LABEL;
     default:
       return INVALID_MESSAGE_ID;
   }
 }
 
-bool IsNewline(AddressField field) {
-  // NEWLINE is an extension for AddressField enum that's used only internally.
-  return field == static_cast<AddressField>(NEWLINE);
-}
-
 }  // namespace
 
 const std::vector<std::string>& GetRegionCodes() {
   return RegionDataConstants::GetRegionCodes();
 }
 
 std::vector<AddressUiComponent> BuildComponents(
     const std::string& region_code,
     const Localization& localization) {
   std::vector<AddressUiComponent> result;
 
   Rule rule;
   rule.CopyFrom(GetDefaultRule());
   if (!rule.ParseSerializedRule(
            RegionDataConstants::GetRegionData(region_code))) {
     return result;
   }
 
-  bool previous_field_is_newline = true;
-  bool next_field_is_newline = true;
-  for (std::vector<AddressField>::const_iterator field_it =
-       rule.GetFormat().begin();
-       field_it != rule.GetFormat().end(); ++field_it) {
-    if (IsNewline(*field_it)) {
-      previous_field_is_newline = true;
-      continue;
+  for (std::vector<std::vector<AddressField> >::const_iterator
+       line_it = rule.GetFormat().begin();
+       line_it != rule.GetFormat().end();
+       ++line_it) {
+    for (std::vector<AddressField>::const_iterator field_it = line_it->begin();
+         field_it != line_it->end(); ++field_it) {
+      AddressUiComponent component;
+      component.length_hint =
+          line_it->size() == 1 ? AddressUiComponent::HINT_LONG

[please use gerrit instead, 2013/12/20 22:03:33]

I like that this logic is now quite clear :-)

[Evan Stade, 2013/12/21 00:27:13]

what happened to SHORT_EOL or whatever we were going to call it?

[please use gerrit instead, 2014/01/02 19:49:39]

I've added a test that verifies that no consecutive rows in address formats have
multiple fields. Therefore, SHORT_EOL is not necessary for libaddressinput. (The
address formats are static data in region_data_constants.cc.)

+                               : AddressUiComponent::HINT_SHORT;
+      component.field = *field_it;
+      component.name = localization.GetString(
+          GetMessageIdForField(*field_it, rule.GetAdminAreaNameMessageId(),
+                              rule.GetPostalCodeNameMessageId()));
+      result.push_back(component);
     }
-    AddressUiComponent component;
-    std::vector<AddressField>::const_iterator next_field_it = field_it + 1;
-    next_field_is_newline =
-        next_field_it == rule.GetFormat().end() || IsNewline(*next_field_it);
-    component.length_hint = previous_field_is_newline && next_field_is_newline
-                                ? AddressUiComponent::HINT_LONG
-                                : AddressUiComponent::HINT_SHORT;
-    previous_field_is_newline = false;
-    component.field = *field_it;
-    component.name = localization.GetString(
-        GetMessageIdForField(*field_it, rule.GetAdminAreaNameMessageId(),
-                             rule.GetPostalCodeNameMessageId()));
-    result.push_back(component);
   }
 
   return result;
 }
 
 }  // namespace addressinput
 }  // namespace i18n