| Index: components/autofill/core/browser/personal_data_manager_unittest.cc
|
| diff --git a/components/autofill/core/browser/personal_data_manager_unittest.cc b/components/autofill/core/browser/personal_data_manager_unittest.cc
|
| index 96e3a83134065287186fac9917f4c2c2a384e60d..2a1b23db52fc1c27949428a1859cfb752853c0ec 100644
|
| --- a/components/autofill/core/browser/personal_data_manager_unittest.cc
|
| +++ b/components/autofill/core/browser/personal_data_manager_unittest.cc
|
| @@ -24,7 +24,6 @@
|
| #include "base/strings/utf_string_conversions.h"
|
| #include "base/synchronization/waitable_event.h"
|
| #include "base/test/histogram_tester.h"
|
| -#include "base/test/scoped_feature_list.h"
|
| #include "base/test/simple_test_clock.h"
|
| #include "base/threading/thread_task_runner_handle.h"
|
| #include "base/time/time.h"
|
| @@ -3636,8 +3635,8 @@ TEST_F(PersonalDataManagerTest,
|
| base::RunLoop().Run();
|
|
|
| std::vector<Suggestion> suggestions =
|
| - personal_data_->GetCreditCardSuggestions(
|
| - AutofillType(CREDIT_CARD_NUMBER), base::ASCIIToUTF16("12345678"));
|
| + personal_data_->GetCreditCardSuggestions(AutofillType(CREDIT_CARD_NUMBER),
|
| + base::ASCIIToUTF16("12345678"));
|
|
|
| // There should be no suggestions.
|
| ASSERT_EQ(0U, suggestions.size());
|
| @@ -3924,74 +3923,6 @@ TEST_F(PersonalDataManagerTest,
|
| ASSERT_EQ(3U, suggestions.size());
|
| }
|
|
|
| -// Tests that server cards will shown bank name when bank name available and
|
| -// feature flag on.
|
| -TEST_F(PersonalDataManagerTest,
|
| - GetCreditCardSuggestions_ShowBankNameOfServerCards) {
|
| - // Turn on feature flag.
|
| - base::test::ScopedFeatureList scoped_feature_list_;
|
| - scoped_feature_list_.InitAndEnableFeature(kAutofillCreditCardBankNameDisplay);
|
| -
|
| - EnableWalletCardImport();
|
| -
|
| - // Add a local card.
|
| - CreditCard credit_card0("287151C8-6AB1-487C-9095-28E80BE5DA15",
|
| - "https://www.example.com");
|
| - test::SetCreditCardInfo(&credit_card0, "Clyde Barrow",
|
| - "347666888555" /* American Express */, "04", "2999",
|
| - "1");
|
| - credit_card0.set_use_count(3);
|
| - credit_card0.set_use_date(AutofillClock::Now() -
|
| - base::TimeDelta::FromDays(1));
|
| - personal_data_->AddCreditCard(credit_card0);
|
| -
|
| - std::vector<CreditCard> server_cards;
|
| -
|
| - // Add a server card without bank name.
|
| - server_cards.push_back(CreditCard(CreditCard::MASKED_SERVER_CARD, "b459"));
|
| - test::SetCreditCardInfo(&server_cards.back(), "Emmet Dalton", "2110", "12",
|
| - "2999", "1");
|
| - server_cards.back().set_use_count(2);
|
| - server_cards.back().set_use_date(AutofillClock::Now() -
|
| - base::TimeDelta::FromDays(1));
|
| - server_cards.back().SetNetworkForMaskedCard(kVisaCard);
|
| -
|
| - // Add a server card with bank name.
|
| - server_cards.push_back(CreditCard(CreditCard::MASKED_SERVER_CARD, "b460"));
|
| - test::SetCreditCardInfo(&server_cards.back(), "Emmet Dalton", "2111", "12",
|
| - "2999", "1");
|
| - server_cards.back().set_use_count(1);
|
| - server_cards.back().set_use_date(AutofillClock::Now() -
|
| - base::TimeDelta::FromDays(1));
|
| - server_cards.back().SetNetworkForMaskedCard(kVisaCard);
|
| - server_cards.back().set_bank_name("Chase");
|
| -
|
| - test::SetServerCreditCards(autofill_table_, server_cards);
|
| -
|
| - EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
|
| - .WillOnce(QuitMainMessageLoop());
|
| - base::RunLoop().Run();
|
| -
|
| - std::vector<Suggestion> suggestions =
|
| - personal_data_->GetCreditCardSuggestions(
|
| - AutofillType(CREDIT_CARD_NUMBER),
|
| - /* field_contents= */ base::string16());
|
| - ASSERT_EQ(3U, suggestions.size());
|
| -
|
| - // Local cards will show network.
|
| - EXPECT_EQ(
|
| - base::UTF8ToUTF16(std::string("Amex") + kUTF8MidlineEllipsis + "8555"),
|
| - suggestions[0].value);
|
| - // Server card without bank name will show network.
|
| - EXPECT_EQ(
|
| - base::UTF8ToUTF16(std::string("Visa") + kUTF8MidlineEllipsis + "2110"),
|
| - suggestions[1].value);
|
| - // Server card with bank name will show bank name.
|
| - EXPECT_EQ(
|
| - base::UTF8ToUTF16(std::string("Chase") + kUTF8MidlineEllipsis + "2111"),
|
| - suggestions[2].value);
|
| -}
|
| -
|
| // Tests that only the full server card is kept when deduping with a local
|
| // duplicate of it.
|
| TEST_F(PersonalDataManagerTest,
|
|
|