PaymentHandler: Implement GetAllPaymentApps().

content/browser/payments/payment_app_provider_impl.cc

 // 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.
 
 #include "content/browser/payments/payment_app_provider_impl.h"
 
 #include "content/browser/payments/payment_app_context_impl.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/browser/service_worker/service_worker_metrics.h"
 #include "content/browser/service_worker/service_worker_version.h"
@@ skipping 55 matching lines @@
 
 void GetAllManifestsOnIO(
     scoped_refptr<PaymentAppContextImpl> payment_app_context,
     const PaymentAppProvider::GetAllManifestsCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   payment_app_context->payment_app_database()->ReadAllManifests(
       base::Bind(&DidGetAllManifestsOnIO, callback));
 }
 
+void DidGetAllPaymentAppsOnIO(
+    PaymentAppProvider::GetAllPaymentAppsCallback callback,
+    PaymentAppProvider::PaymentApps apps) {
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      base::BindOnce(std::move(callback), base::Passed(std::move(apps))));
+}
+
+void GetAllPaymentAppsOnIO(
+    scoped_refptr<PaymentAppContextImpl> payment_app_context,
+    PaymentAppProvider::GetAllPaymentAppsCallback callback) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+
+  payment_app_context->payment_app_database()->ReadAllPaymentApps(
+      base::BindOnce(&DidGetAllPaymentAppsOnIO, std::move(callback)));
+}
+
 void DispatchPaymentRequestEventError(
     ServiceWorkerStatusCode service_worker_status) {
   NOTIMPLEMENTED();
 }
 
 void DispatchPaymentRequestEvent(
     payments::mojom::PaymentAppRequestPtr app_request,
     const PaymentAppProvider::InvokePaymentAppCallback& callback,
     scoped_refptr<ServiceWorkerVersion> active_version) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
@@ skipping 70 matching lines @@
   StoragePartitionImpl* partition = static_cast<StoragePartitionImpl*>(
       BrowserContext::GetDefaultStoragePartition(browser_context));
   scoped_refptr<PaymentAppContextImpl> payment_app_context =
       partition->GetPaymentAppContext();
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&GetAllManifestsOnIO, payment_app_context, callback));
 }
 
+void PaymentAppProviderImpl::GetAllPaymentApps(
+    BrowserContext* browser_context,
+    GetAllPaymentAppsCallback callback) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+
+  StoragePartitionImpl* partition = static_cast<StoragePartitionImpl*>(
+      BrowserContext::GetDefaultStoragePartition(browser_context));

[nasko, 2017/05/09 22:46:44]

Are payments supported in Chrome Apps? If yes, then this is going to break in
those cases.

[zino, 2017/05/10 15:57:42]

This feature shouldn't be supported in Chrome Apps.
This Payment Handler is providing third-party payment processing to merchant
site.
When the Payment Request API is initiated in merchant site, the browser will
display the proper payment instruments and then invoke the payment instrument by
user interaction.
At that time, if the third-party payment app is Chrome App, then the payment app
can get a excessive permission.
So, it can't be Chrome App for security reason.
(In the past CL's comment: https://codereview.chromium.org/2586213002/#msg33)

[nasko, 2017/05/11 21:45:39]

If this is the case, do we block the API in the case of Chrome Apps? Do we have
a test to ensure we won't regress that?

[zino, 2017/05/12 20:59:21]

Oh, I thought that the service worker is only allowed in SecureContext but it's
not true.
Okay I'll follow-up this issue. I think we should block accessing the
PaymentManager in Extension side.
(FYI, the PaymentManager is an extension of SW registration like PushManager.
and it is used to store payment instruments.)

If you don't mind, can I fix this in separated CL?
The change should be in Blink and extension side(api test) as well as has a bit
different content with this patch's title.
I already filed a bug here:
https://bugs.chromium.org/p/chromium/issues/detail?id=721897

Anyway, this feature is still behind runtime flag. So, the PaymentManager is not
exposed.
Also, even if someone enables the feature and then store some instruments, this
patch ignores reading the data (using GetDefaultStoragePartition only).

[nasko, 2017/05/12 21:12:10]

chrome-extension: is likely treated as a secure context.
Yes, separate CL is totally fine, and actually encouraged : ).
Thanks!

+  scoped_refptr<PaymentAppContextImpl> payment_app_context =
+      partition->GetPaymentAppContext();
+
+  BrowserThread::PostTask(
+      BrowserThread::IO, FROM_HERE,
+      base::BindOnce(&GetAllPaymentAppsOnIO, payment_app_context,
+                     std::move(callback)));
+}
+
 void PaymentAppProviderImpl::InvokePaymentApp(
     BrowserContext* browser_context,
     int64_t registration_id,
     payments::mojom::PaymentAppRequestPtr app_request,
     const InvokePaymentAppCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   StoragePartitionImpl* partition = static_cast<StoragePartitionImpl*>(
       BrowserContext::GetDefaultStoragePartition(browser_context));
   scoped_refptr<ServiceWorkerContextWrapper> service_worker_context =
       partition->GetServiceWorkerContext();
 
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&FindRegistrationOnIO, std::move(service_worker_context),
                  registration_id, base::Passed(std::move(app_request)),
                  callback));
 }
 
 PaymentAppProviderImpl::PaymentAppProviderImpl() {}
 
 PaymentAppProviderImpl::~PaymentAppProviderImpl() {}
 
 }  // namespace content