[Payments] Add web app manifest section table in SQLite web database

components/payments/content/BUILD.gn

 # Copyright 2016 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.
 
 import("//mojo/public/tools/bindings/mojom.gni")
 
 static_library("content") {
   sources = [
     "payment_request.cc",
     "payment_request.h",
@@ skipping 43 matching lines @@
 }
 
 static_library("utils") {
   sources = [
     "payment_details_validation.cc",
     "payment_details_validation.h",
     "payment_manifest_downloader.cc",
     "payment_manifest_downloader.h",
     "payment_manifest_parser_host.cc",
     "payment_manifest_parser_host.h",
+    "payment_manifest_section_table.cc",
+    "payment_manifest_section_table.h",
     "payments_validators.cc",
     "payments_validators.h",
   ]
 
   deps = [
     ":mojom",
     ":mojom_parser",
     "//base",
     "//components/autofill/core/browser",
     "//components/data_use_measurement/core",
     "//components/link_header_util",
     "//components/payments/core",
     "//components/strings",
+    "//components/webdata/common:common",

[please use gerrit instead, 2017/04/05 20:33:30]

:common is redundant. Remove.

[gogerald1, 2017/04/05 23:38:48]

common is not the only target in that gn, why it is redundant?

[please use gerrit instead, 2017/04/06 14:35:28]

"//components/webdata/common:common" is equivalent to
"//components/webdata/common". So ":common" is not adding any more information,
it only slows down whoever is reading this file.

[gogerald1, 2017/04/06 16:42:09]

common is the default target of //components/webdata/common, then make it
explicit would be better?

Anyway, addressed it as you commented, saw various preference in gn, but mostly
use implicit target name

     "//content/public/browser",
     "//net",
+    "//sql:sql",

[please use gerrit instead, 2017/04/05 20:33:30]

:sql is redundant. Remove.

[gogerald1, 2017/04/05 23:38:48]

ditto

     "//third_party/re2",
     "//ui/base",
     "//url",
   ]
 
   public_deps = [
     "//third_party/icu",
     "//third_party/libaddressinput",
   ]
 }
 
 source_set("unit_tests") {
   testonly = true
   sources = [
     "payment_manifest_downloader_unittest.cc",
+    "payment_manifest_section_table_unittest.cc",
     "payment_request_spec_unittest.cc",
     "payment_request_state_unittest.cc",
     "payments_validators_unittest.cc",
   ]
 
   deps = [
     ":content",
     ":mojom",
     ":utils",
     "//base",
     "//base/test:test_support",
     "//components/autofill/core/browser",
     "//components/autofill/core/browser:test_support",
     "//content/test:test_support",
     "//net:test_support",
     "//testing/gtest",
     "//third_party/icu",
     "//third_party/libaddressinput:test_support",
   ]
 }