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

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

Issue 2897133002: Revert of [WebPayments] Adding FilterProfilesForShipping to profile comparator (Closed)
Patch Set: 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/profile_list_view_controller_browsertest.cc
diff --git a/chrome/browser/ui/views/payments/profile_list_view_controller_browsertest.cc b/chrome/browser/ui/views/payments/profile_list_view_controller_browsertest.cc
deleted file mode 100644
index 319443f3b1b22344c03368acea70af17df7e29b1..0000000000000000000000000000000000000000
--- a/chrome/browser/ui/views/payments/profile_list_view_controller_browsertest.cc
+++ /dev/null
@@ -1,78 +0,0 @@
-// 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 "base/guid.h"
-#include "base/strings/utf_string_conversions.h"
-#include "chrome/browser/ui/views/payments/payment_request_browsertest_base.h"
-#include "chrome/browser/ui/views/payments/payment_request_dialog_view_ids.h"
-#include "components/autofill/core/browser/autofill_profile.h"
-#include "components/autofill/core/browser/autofill_test_utils.h"
-#include "components/autofill/core/browser/field_types.h"
-#include "components/autofill/core/browser/personal_data_manager.h"
-#include "ui/views/controls/label.h"
-
-namespace payments {
-
-autofill::AutofillProfile CreateProfileWithPartialAddress() {
- autofill::AutofillProfile profile = autofill::test::GetFullProfile2();
- profile.SetRawInfo(autofill::ADDRESS_HOME_LINE1, base::ASCIIToUTF16(""));
- profile.SetRawInfo(autofill::ADDRESS_HOME_LINE2, base::ASCIIToUTF16(""));
- profile.SetRawInfo(autofill::ADDRESS_HOME_CITY, base::ASCIIToUTF16(""));
- profile.SetRawInfo(autofill::ADDRESS_HOME_STATE, base::ASCIIToUTF16(""));
- return profile;
-}
-
-class PaymentRequestProfileListTest : public PaymentRequestBrowserTestBase {
- protected:
- PaymentRequestProfileListTest()
- : PaymentRequestBrowserTestBase(
- "/payment_request_free_shipping_test.html") {}
-};
-
-IN_PROC_BROWSER_TEST_F(PaymentRequestProfileListTest, PrioritizeCompleteness) {
- autofill::AutofillProfile complete = autofill::test::GetFullProfile();
- autofill::AutofillProfile partial = CreateProfileWithPartialAddress();
- partial.set_use_count(1000);
-
- AddAutofillProfile(complete);
- AddAutofillProfile(partial);
-
- // In the Personal Data Manager, the partial address is more frecent.
- autofill::PersonalDataManager* personal_data_manager = GetDataManager();
- std::vector<autofill::AutofillProfile*> profiles =
- personal_data_manager->GetProfiles();
- ASSERT_EQ(2UL, profiles.size());
- EXPECT_EQ(partial, *profiles[0]);
- EXPECT_EQ(complete, *profiles[1]);
-
- InvokePaymentRequestUI();
-
- PaymentRequest* request = GetPaymentRequests(GetActiveWebContents()).front();
-
- // The complete profile should be selected.
- ASSERT_TRUE(request->state()->selected_shipping_profile());
- EXPECT_EQ(complete, *request->state()->selected_shipping_profile());
-
- // It should appear first in the shipping profiles.
- ASSERT_EQ(2UL, request->state()->shipping_profiles().size());
- EXPECT_EQ(complete, *request->state()->shipping_profiles()[0]);
- EXPECT_EQ(partial, *request->state()->shipping_profiles()[1]);
-
- // And both should appear in the UI.
- OpenShippingAddressSectionScreen();
- views::View* sheet = dialog_view()->GetViewByID(
- static_cast<int>(DialogViewID::SHIPPING_ADDRESS_SHEET_LIST_VIEW));
- ASSERT_EQ(2, sheet->child_count());
- views::View* first_label = sheet->child_at(0)->GetViewByID(
- static_cast<int>(DialogViewID::PROFILE_LABEL_LINE_1));
- views::View* second_label = sheet->child_at(1)->GetViewByID(
- static_cast<int>(DialogViewID::PROFILE_LABEL_LINE_1));
-
- EXPECT_EQ(base::ASCIIToUTF16("John H. Doe"),
- static_cast<views::Label*>(first_label)->text());
- EXPECT_EQ(base::ASCIIToUTF16("Jane A. Smith"),
- static_cast<views::Label*>(second_label)->text());
-}
-
-} // namespace payments
« no previous file with comments | « chrome/browser/ui/views/payments/contact_info_editor_view_controller_browsertest.cc ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698