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

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

Issue 2849523003: Add billing address as a mandatory field of Payments credit cards. (Closed)
Patch Set: Moved label creation back to base class Created 3 years, 7 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/shipping_address_editor_view_controller_browsertest.cc
diff --git a/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc b/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
index 77386fec09803595fd98dfa2a1bbda99a0853788..c40e5069cc074d6a3299b6d4b474d006965b7704 100644
--- a/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
+++ b/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
@@ -166,7 +166,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest, SyncData) {
base::RunLoop data_loop;
EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
.WillOnce(QuitMessageLoop(&data_loop));
- ClickOnDialogViewAndWait(DialogViewID::EDITOR_SAVE_BUTTON);
+ ClickOnDialogViewAndWait(DialogViewID::SAVE_ADDRESS_BUTTON);
data_loop.Run();
ASSERT_EQ(1UL, personal_data_manager->GetProfiles().size());
@@ -249,7 +249,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest, AsyncData) {
base::RunLoop data_loop;
EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
.WillOnce(QuitMessageLoop(&data_loop));
- ClickOnDialogViewAndWait(DialogViewID::EDITOR_SAVE_BUTTON);
+ ClickOnDialogViewAndWait(DialogViewID::SAVE_ADDRESS_BUTTON);
data_loop.Run();
ASSERT_EQ(1UL, personal_data_manager->GetProfiles().size());
@@ -389,7 +389,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
base::RunLoop data_loop;
EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
.WillOnce(QuitMessageLoop(&data_loop));
- ClickOnDialogViewAndWait(DialogViewID::EDITOR_SAVE_BUTTON);
+ ClickOnDialogViewAndWait(DialogViewID::SAVE_ADDRESS_BUTTON);
data_loop.Run();
ASSERT_EQ(1UL, personal_data_manager->GetProfiles().size());
@@ -428,7 +428,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
base::RunLoop data_loop;
EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
.WillOnce(QuitMessageLoop(&data_loop));
- ClickOnDialogViewAndWait(DialogViewID::EDITOR_SAVE_BUTTON);
+ ClickOnDialogViewAndWait(DialogViewID::SAVE_ADDRESS_BUTTON);
data_loop.Run();
ASSERT_EQ(1UL, personal_data_manager->GetProfiles().size());
@@ -479,7 +479,7 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
base::RunLoop data_loop;
EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
.WillOnce(QuitMessageLoop(&data_loop));
- ClickOnDialogViewAndWait(DialogViewID::EDITOR_SAVE_BUTTON);
+ ClickOnDialogViewAndWait(DialogViewID::SAVE_ADDRESS_BUTTON);
data_loop.Run();
ExpectExistingRequiredFields(/*unset_types=*/nullptr);

Powered by Google App Engine
This is Rietveld 408576698