Download web payment manifests.

components/payments/BUILD.gn

Index: components/payments/BUILD.gn
diff --git a/components/payments/BUILD.gn b/components/payments/BUILD.gn
index 46f08096fd6effdc963ddfae5aa9cc820d516967..1e4401705d2e452282d7d8982ca3d1f66ca36815 100644
--- a/components/payments/BUILD.gn
+++ b/components/payments/BUILD.gn
@@ -42,7 +42,7 @@ if (!is_ios) {
 
     deps = [
       ":payment_request",
-      ":payment_validation",
+      ":payment_utils",
       "//components/autofill/core/browser",
       "//content/public/browser",
       "//mojo/public/cpp/bindings",
@@ -50,7 +50,7 @@ if (!is_ios) {
   }
 }
 
-static_library("payment_validation") {
+static_library("payment_utils") {
   sources = [
     "address_normalizer.cc",
     "address_normalizer.h",
@@ -58,6 +58,8 @@ static_library("payment_validation") {
     "currency_formatter.h",
     "payment_details_validation.cc",
     "payment_details_validation.h",
+    "payment_manifest_downloader.cc",
+    "payment_manifest_downloader.h",
     "payments_validators.cc",
     "payments_validators.h",
   ]
@@ -66,8 +68,11 @@ static_library("payment_validation") {
     ":payment_request",
     "//base",
     "//components/autofill/core/browser",
-    "//third_party/re2:re2",
-    "//url:url",
+    "//components/data_use_measurement/core",
+    "//components/link_header_util",
+    "//net",
+    "//third_party/re2",
+    "//url",
   ]
 
   public_deps = [
@@ -81,16 +86,20 @@ source_set("unit_tests") {
   sources = [
     "address_normalizer_unittest.cc",
     "currency_formatter_unittest.cc",
+    "payment_manifest_downloader_test.cc",
     "payments_validators_test.cc",
   ]
 
   deps = [
-    ":payment_validation",
+    ":payment_utils",
     "//base",
     "//base/test:test_support",
     "//components/autofill/core/browser",
+    "//content/test:test_support",

[sdefresne, 2017/02/23 17:11:05]

I think the iOS build failures are due to this line.

When gn parse a BUILD.gn file, all targets are evaluated and their dependencies
loaded, even if the target itself is never used on iOS. However, loading
//content is not allowed on iOS, so this cause the failure.

As //components/payments is build and used on iOS, it cannot depends on
//content. You should try to remove this dependency on //content. One option is
to re-implement the helper you use from //content. Another is to turn your
component into a layered component
(https://www.chromium.org/developers/design-documents/layered-components-design).

[please use gerrit instead, 2017/02/23 19:57:50]

Started on splitting up the component into layers in a separate patch, which
will need to land first.

[please use gerrit instead, 2017/03/03 19:50:47]

The split was done in a previous patch. This patch is rebased on top.

+    "//net:test_support",
+    "//testing/gmock",
     "//testing/gtest",
-    "//third_party/icu:icu",
+    "//third_party/icu",
     "//third_party/libaddressinput:test_support",
   ]
 }