PaymentApp: Implement InvokePaymentApp() in browser side.

content/browser/payments/payment_app_content_unittest_base.h

Index: content/browser/payments/payment_app_content_unittest_base.h
diff --git a/content/browser/payments/payment_app_content_unittest_base.h b/content/browser/payments/payment_app_content_unittest_base.h
index b0d9ce9ea2a0d09dc854484355361c74057f1df2..001b5038fdea5dfccc91c06014908917f9e92efb 100644
--- a/content/browser/payments/payment_app_content_unittest_base.h
+++ b/content/browser/payments/payment_app_content_unittest_base.h
@@ -19,7 +19,6 @@
 namespace content {
 
 class BrowserContext;
-class EmbeddedWorkerTestHelper;
 class PaymentAppContextImpl;
 class StoragePartitionImpl;
 class TestBrowserThreadBundle;
@@ -41,12 +40,19 @@ class PaymentAppContentUnitTestBase : public testing::Test {
       const std::string& name);
   void UnregisterServiceWorker(const GURL& scope_url);
 
+  void init_payment_app_invoked() const;

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

Only simple getters and settings can use hacker_case. This method looks like
neither. Therefore, it should use CamelCase.

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

Done.

+  bool payment_app_invoked() const;
+  int64_t last_sw_registration_id() const;
+  const GURL& last_sw_scope_url() const;
+
  private:
   StoragePartitionImpl* storage_partition();
   PaymentAppContextImpl* payment_app_context();
 
+  struct PaymentAppForWorkerTestHelper;

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

Forward declarations should precede method declarations in the "private"
section.

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

Done.

+
   std::unique_ptr<TestBrowserThreadBundle> thread_bundle_;
-  std::unique_ptr<EmbeddedWorkerTestHelper> embedded_worker_helper_;
+  std::unique_ptr<PaymentAppForWorkerTestHelper> worker_helper_;
   std::vector<payments::mojom::PaymentAppManagerPtr> payment_app_managers_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentAppContentUnitTestBase);