Build libaddressinput in Chrome for requestAutocomplete().

third_party/libaddressinput/libaddressinput.gyp

Index: third_party/libaddressinput/libaddressinput.gyp
diff --git a/third_party/libaddressinput/libaddressinput.gyp b/third_party/libaddressinput/libaddressinput.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..dac9e136e5865d06a72759e9c9079b5229b324fb
--- /dev/null
+++ b/third_party/libaddressinput/libaddressinput.gyp
@@ -0,0 +1,92 @@
+# 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.
+{
+  'target_defaults': {
+    'cflags!': [
+      '-fvisibility=hidden',
+    ],
+  },
+  'targets': [
+    {
+      'target_name': 'generated_messages',
+      'type': 'none',
+      'variables': {
+        'grit_out_dir': '<(SHARED_INTERMEDIATE_DIR)/libaddressinput/',
+      },
+      'actions': [
+        {
+          'action_name': 'generate_messages',
+          'variables': {
+            'grit_grd_file': 'src/cpp/res/messages.grd',
+          },
+          'includes': [
+            '../../build/grit_action.gypi',
+          ],
+        },
+      ],
+      'includes': [
+        '../../build/grit_target.gypi',
+      ],
+    },
+    {
+      'target_name': 'libaddressinput',
+      'type': '<(component)',
+      'include_dirs': [
+        # List chromium/ directory first to pick up "util/json.h" as an include.
+        'chromium/',
+        '<(DEPTH)/',
+        'src/cpp/include/',
+        'src/cpp/src/',

[please use gerrit instead, 2013/11/20 00:03:10]

The files in 'cpp/src/src' use quotation marks "" instead of <> when including
the headers from the same directory. I believe that the search path for header
files starts with the current directories when using "" instead of <>. Unless
removing 'src/cpp/src' from the 'include_dirs' breaks json.h, I recommend
removing 'src/cpp/src' from this list.

[Dan Beam, 2013/11/20 00:53:48]

Done.

+        '<(SHARED_INTERMEDIATE_DIR)/libaddressinput/',
+      ],
+      'sources': [
+        'chromium/util/json.cc',
+        'chromium/util/json.h',
+        'src/cpp/include/libaddressinput/address_field.h',
+        'src/cpp/include/libaddressinput/address_ui_component.h',
+        'src/cpp/include/libaddressinput/address_ui.h',
+        'src/cpp/include/libaddressinput/localization.h',
+        'src/cpp/src/address_field.cc',
+        'src/cpp/src/address_field_util.cc',
+        'src/cpp/src/address_ui.cc',
+        'src/cpp/src/localization.cc',
+        'src/cpp/src/region_data_constants.cc',
+        'src/cpp/src/rule.cc',

[please use gerrit instead, 2013/11/20 00:03:10]

If you're including all of the header files in the 'sources' section, then
please also add 'address_field_util.h', 'region_data_constants.h', and 'rule.h'.

[Dan Beam, 2013/11/20 00:53:48]

Done.

+      ],
+      'dependencies': [
+        'generated_messages',
+        '<(DEPTH)/base/base.gyp:base',
+      ],
+      'direct_dependent_settings': {
+        'include_dirs': [
+          'chromium/',
+          '<(DEPTH)/',
+          'src/cpp/include/',
+          'src/cpp/src/',
+          '<(SHARED_INTERMEDIATE_DIR)/libaddressinput/',

[please use gerrit instead, 2013/11/20 00:03:10]

Direct depending settings are settings for the autofill module that will use
libaddressinput. Technically, the autofill module should include only
'src/cpp/include' from libaddressinput. 'chromium', 'src/cpp/src/', and
<(SHARED_INTERMEDIATE_DIR)/libaddressinput/' are implementation details of
libaddressinput. Autofill should not have those includes. '<(DEPTH)' is included
by default in all Chrome code. That's not necessary.

[Dan Beam, 2013/11/20 00:53:48]

Done.

+        ],
+      },
+    },
+    {
+      'target_name': 'libaddressinput_unittests',
+      'type': '<(gtest_target_type)',
+      'include_dirs+': [
+        '<(DEPTH)/testing/gtest/include/',
+      ],
+      'sources': [
+        'src/cpp/test/address_field_util_test.cc',
+        'src/cpp/test/address_ui_test.cc',
+        'src/cpp/test/localization_test.cc',
+        'src/cpp/test/region_data_constants_test.cc',
+        'src/cpp/test/rule_test.cc',
+        'src/cpp/test/util/json_test.cc',
+      ],
+      'dependencies': [
+        'libaddressinput',
+        '<(DEPTH)/base/base.gyp:run_all_unittests',

[please use gerrit instead, 2013/11/20 00:03:10]

'libaddressinput_unittests' depends on 'run_all_unittests'? I think you meant
the other way around.

[Dan Beam, 2013/11/20 00:53:48]

look up run_all_unittests

+        '<(DEPTH)/testing/gtest.gyp:gtest',
+      ],
+    },
+  ],
+}