[Payments] Cache payment manifests.

components/payments/android/payment_manifest_web_data_service.h

Index: components/payments/android/payment_manifest_web_data_service.h
diff --git a/components/payments/android/payment_manifest_web_data_service.h b/components/payments/android/payment_manifest_web_data_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..7eaf537022020f401b4f76ec2a13ce5adc4c01ee
--- /dev/null
+++ b/components/payments/android/payment_manifest_web_data_service.h
@@ -0,0 +1,68 @@
+// Copyright 2017 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.
+
+#ifndef COMPONENTS_PAYMENTS_ANDROID_PAYMENT_MANIFEST_WEB_DATA_SERVICE_H_
+#define COMPONENTS_PAYMENTS_ANDROID_PAYMENT_MANIFEST_WEB_DATA_SERVICE_H_
+
+#include "base/memory/ref_counted.h"
+#include "components/payments/android/payment_method_manifest_table.h"
+#include "components/payments/android/web_app_manifest_section_table.h"
+#include "components/webdata/common/web_data_results.h"
+#include "components/webdata/common/web_data_service_base.h"
+#include "components/webdata/common/web_data_service_consumer.h"
+#include "components/webdata/common/web_database_service.h"
+
+namespace payments {
+
+// Web data service to read/write data in WebAppManifestSectionTable and
+// PaymentMethodManifestTable.
+class PaymentManifestWebDataService : public WebDataServiceBase {
+ public:
+  PaymentManifestWebDataService(
+      scoped_refptr<WebDatabaseService> wdbs,
+      const ProfileErrorCallback& callback,
+      const scoped_refptr<base::SingleThreadTaskRunner>& ui_thread);
+
+  // Adds the web app |manifest|.
+  void AddPaymentWebAppManifest(
+      std::vector<mojom::WebAppManifestSectionPtr> manifest);

[please use gerrit instead, 2017/04/26 15:00:44]

Good that you've removed const-ref here, because mojom::WebAppManifestSectionPtr
is a move-only type.

[gogerald1, 2017/04/26 17:30:31]

Acknowledged.

+
+  // Adds the |payment_method|'s manifest.
+  void AddPaymentMethodManifest(const std::string& payment_method,
+                                std::vector<std::string> apps_package_names);

[please use gerrit instead, 2017/04/26 15:00:44]

Please use const-ref for apps_package_names, otherwise std::vector<std::string>
will be inefficiently copied.

[gogerald1, 2017/04/26 17:30:31]

There is no copy here since I used std::move when calling
AddPaymentMethodManifest and inside AddPaymentMethodManifest function. I did
this to make it efficient and consistent with AddPaymentWebAppManifest.

+
+  // Gets the |web_app|'s manifest.
+  WebDataServiceBase::Handle GetPaymentWebAppManifest(
+      const std::string& web_app,
+      WebDataServiceConsumer* consumer);
+
+  // Gets the |payment_method|'s manifest.
+  WebDataServiceBase::Handle GetPaymentMethodManifest(
+      const std::string& payment_method,
+      WebDataServiceConsumer* consumer);
+
+ private:
+  ~PaymentManifestWebDataService() override;
+
+  WebDatabase::State AddPaymentWebAppManifestImpl(
+      const std::vector<mojom::WebAppManifestSectionPtr>& manifest,
+      WebDatabase* db);
+  WebDatabase::State AddPaymentMethodManifestImpl(
+      const std::string& payment_method,
+      const std::vector<std::string>& apps_package_names,
+      WebDatabase* db);
+
+  std::unique_ptr<WDTypedResult> GetPaymentWebAppManifestImpl(
+      const std::string& web_app,
+      WebDatabase* db);
+  std::unique_ptr<WDTypedResult> GetPaymentMethodManifestImpl(
+      const std::string& payment_method,
+      WebDatabase* db);
+
+  DISALLOW_COPY_AND_ASSIGN(PaymentManifestWebDataService);
+};
+
+}  // namespace payments
+
+#endif  // COMPONENTS_PAYMENTS_ANDROID_PAYMENT_MANIFEST_WEB_DATA_SERVICE_H_