Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6923)

Unified Diff: chrome/browser/ui/views/payments/payment_request_can_make_payment_browsertest.cc

Issue 2772103002: [Payments] Desktop: never return false in incognito mode for canMakePayment (Closed)
Patch Set: Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 31836bf58fe5feb2ab68723ec8ddf7d979f67315..ca68e55eb9932f62a5ca66d68758bb82ca4d408d 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
@@ -39,6 +39,20 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryTest,
ExpectBodyContains(std::vector<base::string16>{base::ASCIIToUTF16("true")});
}
+// Visa is required, and user has a visa instrument, and user is in incognito
+// mode.
+IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryTest,
+ CanMakePayment_Supported_Incognito) {
+ SetIncognitoForTesting();
+
+ const autofill::CreditCard card = autofill::test::GetCreditCard(); // Visa.
+ AddCreditCard(card);
+
+ CallCanMakePayment();
+
+ ExpectBodyContains(std::vector<base::string16>{base::ASCIIToUTF16("true")});
+}
+
// Visa is required, and user doesn't have a visa instrument.
IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryTest,
CanMakePayment_NotSupported) {
@@ -50,4 +64,20 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryTest,
ExpectBodyContains(std::vector<base::string16>{base::ASCIIToUTF16("false")});
}
+// Visa is required, and user doesn't have a visa instrument and the user is in
+// incognito mode.
+IN_PROC_BROWSER_TEST_F(PaymentRequestCanMakePaymentQueryTest,
+ CanMakePayment_NotSupported_Incognito) {
+ SetIncognitoForTesting();
+
+ 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(std::vector<base::string16>{base::ASCIIToUTF16("true")});
+}
+
} // namespace payments

Powered by Google App Engine
This is Rietveld 408576698