PaymentRequest.canMakePayment() query quota on desktop.

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

Index: chrome/browser/ui/views/payments/payment_request_can_make_payment_browsertest.cc
diff --git a/chrome/browser/ui/views/payments/payment_request_can_make_payment_browsertest.cc b/chrome/browser/ui/views/payments/payment_request_can_make_payment_browsertest.cc
index cb4e07299d5955c077d89a0a446acf1f64efd3bf..20611619a8095f61ce57436861ca8ce56c89243b 100644
--- a/chrome/browser/ui/views/payments/payment_request_can_make_payment_browsertest.cc
+++ b/chrome/browser/ui/views/payments/payment_request_can_make_payment_browsertest.cc
@@ -96,4 +96,34 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryTest,
   ExpectBodyContains({"true"});
 }
 
+class PaymentRequestCanMakePaymentQueryCCTest
+    : public PaymentRequestBrowserTestBase {
+ protected:
+  PaymentRequestCanMakePaymentQueryCCTest()
+      : PaymentRequestBrowserTestBase(
+            "/payment_request_can_make_payment_query_cc_test.html") {}
+
+  void CallCanMakePayment(bool visa) {
+    ResetEventObserver(DialogEvent::CAN_MAKE_PAYMENT_CALLED);
+    ASSERT_TRUE(content::ExecuteScript(GetActiveWebContents(),
+                                       visa ? "buy();" : "other_buy();"));
+    WaitForObservedEvent();
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(PaymentRequestCanMakePaymentQueryCCTest);
+};
+
+IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryCCTest, QueryQuota) {

[Mathieu, 2017/05/08 15:28:11]

could we have a test that first queries for basic-card/visa, and then queries
basic-card/visa+mastercard, to see that it behaves correctly. 

[please use gerrit instead, 2017/05/08 17:28:47]

Done.

+  CallCanMakePayment(true /* visa */);

[Mathieu, 2017/05/08 15:28:11]

/*visa=*/true

[please use gerrit instead, 2017/05/08 17:28:47]

Done.

+  ExpectBodyContains({"false"});

[Mathieu, 2017/05/08 15:28:11]

// User doesn't have a visa.

[please use gerrit instead, 2017/05/08 17:28:47]

Done.

+  CallCanMakePayment(false /* mastercard */);

[Mathieu, 2017/05/08 15:28:11]

/*visa=*/false);  // Will query for Mastercard.

[please use gerrit instead, 2017/05/08 17:28:47]

Done.

+  ExpectBodyContains({"NotAllowedError"});
+  AddCreditCard(autofill::test::GetCreditCard());  // visa
+  CallCanMakePayment(true /* visa */);
+  ExpectBodyContains({"true"});

[Mathieu, 2017/05/08 15:28:11]

At first read I would have expected "false" to be cached, but then looking at
the code I see that the merchant is allowed to repeat an exact query and that
the reply can change. Let's add a comment about that here?

[please use gerrit instead, 2017/05/08 17:28:47]

Done.

+  CallCanMakePayment(false /* mastercard */);
+  ExpectBodyContains({"NotAllowedError"});
+}
+
 }  // namespace payments