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

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

Issue 2854973002: [Web Payments] Go to editors directly when "Add" buttons are pressed (Closed)
Patch Set: Fix comments. Created 3 years, 8 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/credit_card_editor_view_controller_browsertest.cc
diff --git a/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc b/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
index 245da2bab9680b788092f57bdb7e1857e564a8f3..4fc96048e44eb063707370d04d37059d0cca5a4e 100644
--- a/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
+++ b/chrome/browser/ui/views/payments/credit_card_editor_view_controller_browsertest.cc
@@ -54,8 +54,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest, EnteringValidData) {
EXPECT_EQ(0U, request->state()->available_instruments().size());
EXPECT_EQ(nullptr, request->state()->selected_instrument());
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
SetEditorTextfieldValue(base::ASCIIToUTF16("Bob Jones"),
@@ -97,8 +95,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest, EnteringValidData) {
IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest, CancelFromEditor) {
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
ResetEventObserver(DialogEvent::DIALOG_CLOSED);
@@ -114,8 +110,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest,
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
SetEditorTextfieldValue(base::ASCIIToUTF16("Bob Jones"),
@@ -147,8 +141,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest,
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
// This field is required. Entering nothing and blurring out will show
@@ -183,8 +175,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest,
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
SetEditorTextfieldValue(base::ASCIIToUTF16("Bob Jones"),
@@ -216,8 +206,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest,
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
SetEditorTextfieldValue(base::ASCIIToUTF16("Bob Jones"),
@@ -251,8 +239,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest,
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
SetEditorTextfieldValue(base::ASCIIToUTF16("Bob Jones"),
@@ -374,8 +360,6 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest, EditingExpiredCard) {
IN_PROC_BROWSER_TEST_F(PaymentRequestCreditCardEditorTest, EnteringEmptyData) {
InvokePaymentRequestUI();
- OpenPaymentMethodScreen();
-
OpenCreditCardEditorScreen();
// Setting empty data and unfocusing a required textfield will make it

Powered by Google App Engine
This is Rietveld 408576698