[Payments] Implement openWindow for service worker based payment handler

content/browser/payments/payment_app_browsertest.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 "base/command_line.h"
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "components/payments/mojom/payment_app.mojom.h"
 #include "content/browser/storage_partition_impl.h"
 #include "content/public/browser/browser_context.h"
@@ skipping 66 matching lines @@
     EXPECT_TRUE(RunScript("resultQueue.pop()", &script_result));
     return script_result;
   }
 
   void RegisterPaymentApp() {
     std::string script_result;
     ASSERT_TRUE(RunScript("registerPaymentApp()", &script_result));
     ASSERT_EQ("registered", script_result);
   }
 
-  std::vector<int64_t> GetAllPaymentAppIDs() {
+  std::map<std::string, int64_t> GetAllPaymentInstrumentRegistrationIDs() {
     base::RunLoop run_loop;
     PaymentAppProvider::PaymentApps apps;
     PaymentAppProvider::GetInstance()->GetAllPaymentApps(
         shell()->web_contents()->GetBrowserContext(),
         base::BindOnce(&GetAllPaymentAppsCallback, run_loop.QuitClosure(),
                        &apps));
     run_loop.Run();
 
-    std::vector<int64_t> ids;
+    std::map<std::string, int64_t> registrationIds;
     for (const auto& app_info : apps) {
       for (const auto& instrument : app_info.second) {
-        ids.push_back(instrument->registration_id);
+        registrationIds.insert(std::pair<std::string, int64_t>(
+            instrument->instrument_key, instrument->registration_id));
       }
     }
 
-    return ids;
+    return registrationIds;
   }
 
-  PaymentAppResponsePtr InvokePaymentAppWithTestData(int64_t registration_id) {
+  PaymentAppResponsePtr InvokePaymentAppWithTestData(
+      int64_t registration_id,
+      std::string supported_method,
+      std::string instrument_key) {

[please use gerrit instead, 2017/06/02 03:43:16]

const-ref?

[gogerald1, 2017/06/02 13:21:39]

Done.

     PaymentAppRequestPtr app_request = PaymentAppRequest::New();
 
     app_request->top_level_origin = GURL("https://example.com");
 
     app_request->payment_request_origin = GURL("https://example.com");
 
     app_request->payment_request_id = "payment-request-id";
 
     app_request->method_data.push_back(PaymentMethodData::New());
-    app_request->method_data[0]->supported_methods = {"basic-card"};
+    app_request->method_data[0]->supported_methods = {supported_method};
 
     app_request->total = PaymentItem::New();
     app_request->total->amount = PaymentCurrencyAmount::New();
     app_request->total->amount->currency = "USD";
 
     PaymentDetailsModifierPtr modifier = PaymentDetailsModifier::New();
     modifier->total = PaymentItem::New();
     modifier->total->amount = PaymentCurrencyAmount::New();
     modifier->total->amount->currency = "USD";
     modifier->method_data = PaymentMethodData::New();
-    modifier->method_data->supported_methods = {"basic-card"};
+    modifier->method_data->supported_methods = {supported_method};
     app_request->modifiers.push_back(std::move(modifier));
 
-    app_request->instrument_key = "instrument-key";
+    app_request->instrument_key = instrument_key;
 
     base::RunLoop run_loop;
     PaymentAppResponsePtr response;
     PaymentAppProvider::GetInstance()->InvokePaymentApp(
         shell()->web_contents()->GetBrowserContext(), registration_id,
         std::move(app_request),
         base::Bind(&InvokePaymentAppCallback, run_loop.QuitClosure(),
                    &response));
     run_loop.Run();
 
@@ skipping 18 matching lines @@
 
  private:
   std::unique_ptr<net::EmbeddedTestServer> https_server_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentAppBrowserTest);
 };
 
 IN_PROC_BROWSER_TEST_F(PaymentAppBrowserTest, PaymentAppInvocation) {
   RegisterPaymentApp();
 
-  std::vector<int64_t> ids = GetAllPaymentAppIDs();
-  ASSERT_EQ(1U, ids.size());
+  std::map<std::string, int64_t> registrationIds =
+      GetAllPaymentInstrumentRegistrationIDs();
+  ASSERT_EQ(2U, registrationIds.size());
 
-  PaymentAppResponsePtr response(InvokePaymentAppWithTestData(ids[0]));
+  PaymentAppResponsePtr response(InvokePaymentAppWithTestData(
+      registrationIds.at("basic-card-payment-app-id"), "basic-card",
+      "basic-card-payment-app-id"));
   ASSERT_EQ("test", response->method_name);
 
   ClearStoragePartitionData();
 
-  ids = GetAllPaymentAppIDs();
-  ASSERT_EQ(0U, ids.size());
+  registrationIds = GetAllPaymentInstrumentRegistrationIDs();
+  ASSERT_EQ(0U, registrationIds.size());
 
   EXPECT_EQ("https://example.com/", PopConsoleString() /* topLevelOrigin */);
   EXPECT_EQ("https://example.com/",
             PopConsoleString() /* paymentRequestOrigin */);
   EXPECT_EQ("payment-request-id", PopConsoleString() /* paymentRequestId */);
   EXPECT_EQ("[{\"supportedMethods\":[\"basic-card\"]}]",
             PopConsoleString() /* methodData */);
   EXPECT_EQ(
       "{\"amount\":{\"currency\":\"USD\",\"currencySystem\":\"urn:iso:std:iso:"
       "4217\",\"value\":\"\"},\"label\":\"\",\"pending\":false}",
       PopConsoleString() /* total */);
   EXPECT_EQ(
       "[{\"additionalDisplayItems\":[],\"supportedMethods\":[\"basic-card\"],"
       "\"total\":{\"amount\":{\"currency\":\"USD\",\"currencySystem\":\"urn:"
       "iso:std:iso:4217\",\"value\":\"\"},\"label\":\"\",\"pending\":false}}]",
       PopConsoleString() /* modifiers */);
-  EXPECT_EQ("instrument-key", PopConsoleString() /* instrumentKey */);
+  EXPECT_EQ("basic-card-payment-app-id",
+            PopConsoleString() /* instrumentKey */);
 }
 
+IN_PROC_BROWSER_TEST_F(PaymentAppBrowserTest, PaymentAppOpenWindowFailed) {
+  RegisterPaymentApp();
+
+  std::map<std::string, int64_t> registrationIds =
+      GetAllPaymentInstrumentRegistrationIDs();
+  ASSERT_EQ(2U, registrationIds.size());
+
+  PaymentAppResponsePtr response(InvokePaymentAppWithTestData(
+      registrationIds.at("bobpay-payment-app-id"), "https://bobpay.com",
+      "bobpay-payment-app-id"));
+  // InvokePaymentAppCallback returns empty method_name in case of failure, like
+  // in PaymentRequestRespondWithObserver::OnResponseRejected.
+  ASSERT_EQ("", response->method_name);
+
+  ClearStoragePartitionData();
+
+  registrationIds = GetAllPaymentInstrumentRegistrationIDs();
+  ASSERT_EQ(0U, registrationIds.size());
+
+  EXPECT_EQ("https://example.com/", PopConsoleString() /* topLevelOrigin */);
+  EXPECT_EQ("https://example.com/",
+            PopConsoleString() /* paymentRequestOrigin */);
+  EXPECT_EQ("payment-request-id", PopConsoleString() /* paymentRequestId */);
+  EXPECT_EQ("[{\"supportedMethods\":[\"https://bobpay.com\"]}]",
+            PopConsoleString() /* methodData */);
+  EXPECT_EQ(
+      "{\"amount\":{\"currency\":\"USD\",\"currencySystem\":\"urn:iso:std:iso:"
+      "4217\",\"value\":\"\"},\"label\":\"\",\"pending\":false}",
+      PopConsoleString() /* total */);
+  EXPECT_EQ(
+      "[{\"additionalDisplayItems\":[],\"supportedMethods\":[\"https://"
+      "bobpay.com\"],"
+      "\"total\":{\"amount\":{\"currency\":\"USD\",\"currencySystem\":\"urn:"
+      "iso:std:iso:4217\",\"value\":\"\"},\"label\":\"\",\"pending\":false}}]",
+      PopConsoleString() /* modifiers */);
+  EXPECT_EQ("bobpay-payment-app-id", PopConsoleString() /* instrumentKey */);
+}
 }  // namespace content