[Payments] Introduce basic interactive browsertests for Payment Request

chrome/browser/ui/views/payments/payment_request_interactive_uitest.cc

Index: chrome/browser/ui/views/payments/payment_request_interactive_uitest.cc
diff --git a/chrome/browser/ui/views/payments/payment_request_interactive_uitest.cc b/chrome/browser/ui/views/payments/payment_request_interactive_uitest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f4838628d8324b77a1c9122b0468a8a360a4ba18
--- /dev/null
+++ b/chrome/browser/ui/views/payments/payment_request_interactive_uitest.cc
@@ -0,0 +1,39 @@
+// 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 <vector>
+
+#include "chrome/browser/ui/views/payments/payment_request_interactive_uitest_base.h"
+#include "components/payments/payment_request.h"
+#include "components/payments/payment_request_web_contents_manager.h"
+
+namespace payments {
+
+class PaymentRequestWebContentsManagerTest
+    : public PaymentRequestInteractiveTestBase {
+ protected:
+  PaymentRequestWebContentsManagerTest()
+      : PaymentRequestInteractiveTestBase(
+            "/payment_request_multiple_requests.html") {}
+};

[please use gerrit instead, 2017/01/17 18:57:20]

Destructor?

[Mathieu, 2017/01/17 19:23:06]

Is there a particular reason I should have one here? I would think the base
class destructor is fine?

[please use gerrit instead, 2017/01/17 20:41:24]

If you don't specify a destructor with "override" keyword, then the parent
destructor does not get called, I think.

[Mathieu, 2017/01/17 20:57:47]

I put a log statement inside ~PaymentRequestInteractiveTestBase() and it is
called for the two tests in this file.

+
+// If the page creates multiple PaymentRequest objects, it should not crash.
+IN_PROC_BROWSER_TEST_F(PaymentRequestWebContentsManagerTest, MultipleRequests) {
+  const std::vector<PaymentRequest*> payment_requests =
+      GetPaymentRequests(GetActiveWebContents());
+  EXPECT_EQ(5U, payment_requests.size());
+}
+
+class PaymentRequestNoShippingTest : public PaymentRequestInteractiveTestBase {
+ protected:
+  PaymentRequestNoShippingTest()
+      : PaymentRequestInteractiveTestBase(
+            "/payment_request_no_shipping_test.html") {}

[please use gerrit instead, 2017/01/17 18:57:20]

Destructor?

[Mathieu, 2017/01/17 19:23:06]

see above

+};
+
+IN_PROC_BROWSER_TEST_F(PaymentRequestNoShippingTest, OpenPaymentRequestSheet) {
+  InvokePaymentRequestUI();
+}
+
+}  // namespace payments