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
 
+struct PaymentAppContentUnitTestBase::PaymentAppForWorkerTestHelper

[zino, 2017/01/11 20:06:43]

@rouslan,

Should this struct be located to below last_sw_scope_url()?
(I'm not sure if it is a good way for readability. In header file, it was anyway
declared in private field.)

[please use gerrit instead, 2017/01/12 19:04:00]

It feels strange to derive a struct from a class. Let's make
PaymentAppForWorkerTestHelper into a class as well.

[please use gerrit instead, 2017/01/12 19:04:00]

Good question. Let's follow the rule of thumb that everything in .cc should be
in the same order as everything .h. I advised in .h that this struct should be
above storage_partition() declaration. Therefore, let's place the struct in .cc
file immediately above the storage_partition() definition.

[zino, 2017/01/13 16:41:10]

Done.

[zino, 2017/01/16 15:01:09]

I tried it but I couldn't because it is used in PaymentAppUnitTestBase()
constructor.

+    : public EmbeddedWorkerTestHelper {
+  PaymentAppForWorkerTestHelper()
+      : EmbeddedWorkerTestHelper(base::FilePath()) {}

[please use gerrit instead, 2017/01/12 19:04:00]

Need to initialize was_dispatched and last_sw_registration_id. Otherwise they
will have random values that will lead to flakes.

[please use gerrit instead, 2017/01/12 19:04:00]

Need to override the parent destructor.

[zino, 2017/01/13 16:41:10]

Done.

+
+  void OnStartWorker(int embedded_worker_id,
+                     int64_t service_worker_version_id,
+                     const GURL& scope,
+                     const GURL& script_url,
+                     bool pause_after_download) override {
+    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);
+  }
+
+  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;

[please use gerrit instead, 2017/01/12 19:04:00]

private:
DISALLOW_COPY_AND_ASSIGN(PaymentAppForWorkerTestHelper);

[zino, 2017/01/13 16:41:10]

Done.

+};
+
 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::init_payment_app_invoked() const {
+  worker_helper_->was_dispatched = false;

[please use gerrit instead, 2017/01/12 19:04:00]

Initialization should happen in the constructor of the worker_helper_. If you
need to reset this value multiple times, please use the term "reset" instead of
"init".

[zino, 2017/01/13 16:41:10]

Done.

+}
+
+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