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

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

Issue 2841643002: When a new Payments address is created from the editor, use it as the chosen address (Closed)
Patch Set: And yet another small goof fix... :-( 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/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 575751428e203f6339b0442bbc7b096bc584c11f..704c45e869418886937278cffc8862741e8e12b5 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
@@ -143,6 +143,11 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest, SyncData) {
SetRegionDataLoader(&test_region_data_loader_);
OpenShippingAddressSectionScreen();
+ // No shipping profiles are available.
+ PaymentRequest* request = GetPaymentRequests(GetActiveWebContents()).front();
+ EXPECT_EQ(0U, request->state()->shipping_profiles().size());
+ EXPECT_EQ(nullptr, request->state()->selected_shipping_profile());
+
test_region_data_loader_.set_synchronous_callback(true);
OpenShippingAddressEditorScreen();
@@ -212,6 +217,12 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest, AsyncData) {
EXPECT_EQ(base::ASCIIToUTF16(kAnyState),
profile->GetRawInfo(autofill::ADDRESS_HOME_STATE));
ExpectExistingRequiredFields(nullptr);
+
+ // One shipping profile is available and selected.
+ PaymentRequest* request = GetPaymentRequests(GetActiveWebContents()).front();
+ EXPECT_EQ(1UL, request->state()->shipping_profiles().size());
+ EXPECT_EQ(request->state()->shipping_profiles().back(),
+ request->state()->selected_shipping_profile());
}
IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,

Powered by Google App Engine
This is Rietveld 408576698