PaymentApp: Implement InvokePaymentApp() in browser side.

content/browser/payments/payment_app_content_unittest_base.cc

 // 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.
 
 #include "content/browser/payments/payment_app_content_unittest_base.h"
 
 #include <stdint.h>
 
 #include <set>
 #include <utility>
 
 #include "base/files/file_path.h"
 #include "base/run_loop.h"
 #include "base/stl_util.h"
 #include "content/browser/service_worker/embedded_worker_test_helper.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/browser/storage_partition_impl.h"
+#include "content/common/service_worker/service_worker_event_dispatcher.mojom.h"
 #include "content/common/service_worker/service_worker_status_code.h"
 #include "content/public/test/test_browser_context.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "mojo/public/cpp/bindings/associated_interface_ptr.h"
 #include "mojo/public/cpp/bindings/interface_request.h"
 
 namespace content {
 
 namespace {
 
 void RegisterServiceWorkerCallback(bool* called,
                                    ServiceWorkerStatusCode status,
                                    const std::string& status_message,
                                    int64_t registration_id) {
   EXPECT_EQ(SERVICE_WORKER_OK, status) << ServiceWorkerStatusToString(status);
   *called = true;
 }
 
 void UnregisterServiceWorkerCallback(bool* called,
                                      ServiceWorkerStatusCode status) {
   EXPECT_EQ(SERVICE_WORKER_OK, status) << ServiceWorkerStatusToString(status);
   *called = true;
 }
 
 }  // namespace
 
+class PaymentAppContentUnitTestBase::PaymentAppForWorkerTestHelper
+    : public EmbeddedWorkerTestHelper {
+ public:
+  PaymentAppForWorkerTestHelper()
+      : EmbeddedWorkerTestHelper(base::FilePath()),
+        was_dispatched(false),
+        last_sw_registration_id(-1) {}

[nhiroki, 2017/01/18 03:47:39]

Can you replace -1 with kInvalidServiceWorkerRegistrationId?
https://chromium.googlesource.com/chromium/src/+/756089ad4639bb917faac4b2f6a8...

[zino, 2017/01/18 15:56:07]

Done.

+  ~PaymentAppForWorkerTestHelper() override {}
+
+  void OnStartWorker(
+      int embedded_worker_id,
+      int64_t service_worker_version_id,
+      const GURL& scope,
+      const GURL& script_url,
+      bool pause_after_download,
+          mojom::ServiceWorkerEventDispatcherRequest request) override {

[nhiroki, 2017/01/18 03:47:39]

This indent is broken.

(We might want to have a presubmit check for code format like this:
https://cs.chromium.org/chromium/src/content/browser/service_worker/PRESUBMIT.py)

[zino, 2017/01/18 15:56:07]

Done.

+    ServiceWorkerVersion* version =
+        context()->GetLiveVersion(service_worker_version_id);
+    version->SetMainScriptHttpResponseInfo(
+        EmbeddedWorkerTestHelper::CreateHttpResponseInfo());
+    last_sw_registration_id = version->registration_id();
+    last_sw_scope = scope;
+    EmbeddedWorkerTestHelper::OnStartWorker(
+        embedded_worker_id, service_worker_version_id, scope, script_url,
+        pause_after_download, std::move(request));
+  }
+
+  void OnPaymentRequestEvent(
+      payments::mojom::PaymentAppRequestDataPtr data,
+      const mojom::ServiceWorkerEventDispatcher::
+          DispatchPaymentRequestEventCallback& callback) override {
+    ASSERT_FALSE(was_dispatched);
+    EmbeddedWorkerTestHelper::OnPaymentRequestEvent(std::move(data), callback);
+    was_dispatched = true;
+  }
+
+  bool was_dispatched;
+  int64_t last_sw_registration_id;
+  GURL last_sw_scope;

[nhiroki, 2017/01/18 03:47:39]

These fields need "_" suffix.

[zino, 2017/01/18 15:56:07]

Done.

+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(PaymentAppForWorkerTestHelper);
+};
+
 PaymentAppContentUnitTestBase::PaymentAppContentUnitTestBase()
     : thread_bundle_(
           new TestBrowserThreadBundle(TestBrowserThreadBundle::IO_MAINLOOP)),
-      embedded_worker_helper_(new EmbeddedWorkerTestHelper(base::FilePath())) {
-  embedded_worker_helper_->context_wrapper()->set_storage_partition(
-      storage_partition());
-  payment_app_context()->Init(embedded_worker_helper_->context_wrapper());
+      worker_helper_(new PaymentAppForWorkerTestHelper()) {
+  worker_helper_->context_wrapper()->set_storage_partition(storage_partition());
+  storage_partition()->service_worker_context_->Shutdown();
+  base::RunLoop().RunUntilIdle();
+
+  storage_partition()->service_worker_context_ =
+      worker_helper_->context_wrapper();
+  payment_app_context()->Init(worker_helper_->context_wrapper());
   base::RunLoop().RunUntilIdle();
 }
 
 PaymentAppContentUnitTestBase::~PaymentAppContentUnitTestBase() {}
 
 BrowserContext* PaymentAppContentUnitTestBase::browser_context() {
-  DCHECK(embedded_worker_helper_);
-  return embedded_worker_helper_->browser_context();
+  DCHECK(worker_helper_);
+  return worker_helper_->browser_context();
 }
 
 PaymentAppManager* PaymentAppContentUnitTestBase::CreatePaymentAppManager(
     const GURL& scope_url,
     const GURL& sw_script_url) {
   // Register service worker for payment app manager.
   bool called = false;
-  embedded_worker_helper_->context()->RegisterServiceWorker(
+  worker_helper_->context()->RegisterServiceWorker(
       scope_url, sw_script_url, nullptr,
       base::Bind(&RegisterServiceWorkerCallback, &called));
   base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(called);
 
   // This function should eventually return created payment app manager
   // but there is no way to get last created payment app manager from
   // payment_app_context()->payment_app_managers_ because its type is std::map
   // and can not ensure its order. So, just make a set of existing payment app
   // managers before creating a new manager and then check what is a new thing.
@@ skipping 58 matching lines @@
   manifest->name = name;
   manifest->options.push_back(std::move(option));
 
   return manifest;
 }
 
 void PaymentAppContentUnitTestBase::UnregisterServiceWorker(
     const GURL& scope_url) {
   // Unregister service worker.
   bool called = false;
-  embedded_worker_helper_->context()->UnregisterServiceWorker(
+  worker_helper_->context()->UnregisterServiceWorker(
       scope_url, base::Bind(&UnregisterServiceWorkerCallback, &called));
   base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(called);
 }
 
+void PaymentAppContentUnitTestBase::ResetPaymentAppInvoked() const {
+  worker_helper_->was_dispatched = false;
+}
+
+bool PaymentAppContentUnitTestBase::payment_app_invoked() const {
+  return worker_helper_->was_dispatched;
+}
+
+int64_t PaymentAppContentUnitTestBase::last_sw_registration_id() const {
+  return worker_helper_->last_sw_registration_id;
+}
+
+const GURL& PaymentAppContentUnitTestBase::last_sw_scope_url() const {
+  return worker_helper_->last_sw_scope;
+}
+
 StoragePartitionImpl* PaymentAppContentUnitTestBase::storage_partition() {
   return static_cast<StoragePartitionImpl*>(
       BrowserContext::GetDefaultStoragePartition(browser_context()));
 }
 
 PaymentAppContextImpl* PaymentAppContentUnitTestBase::payment_app_context() {
   return storage_partition()->GetPaymentAppContext();
 }
 
 }  // namespace content