PaymentRequest.canMakePayment() query quota on desktop.

chrome/browser/ui/views/payments/payment_request_can_make_payment_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 <vector>
 
 #include "base/macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/views/payments/payment_request_browsertest_base.h"
 #include "components/autofill/core/browser/autofill_test_utils.h"
@@ skipping 78 matching lines @@
   const autofill::CreditCard card = autofill::test::GetCreditCard2();  // Amex.
   AddCreditCard(card);
 
   CallCanMakePayment();
 
   // Returns true because the user is in incognito mode, even though it should
   // return false in a normal profile.
   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