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

Unified Diff: components/autofill/core/browser/personal_data_manager_unittest.cc

Issue 1785923010: Remove kAutofillWalletSyncExperimentEnabled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Mark wallet_sync as obsolete, remove wallet_sync_enabled. Created 4 years, 9 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: 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 9964f7a126b79342efd78a3a3ac7aba9686b129b..11f60fb8c1d32dac8b330fb05d282c881a9a4a3f 100644
--- a/components/autofill/core/browser/personal_data_manager_unittest.cc
+++ b/components/autofill/core/browser/personal_data_manager_unittest.cc
@@ -167,7 +167,6 @@ class PersonalDataManagerTest : public testing::Test {
}
void EnableWalletCardImport() {
- prefs_->SetBoolean(prefs::kAutofillWalletSyncExperimentEnabled, true);
signin_manager_->SetAuthenticatedAccountInfo("12345",
"syncuser@example.com");
base::CommandLine::ForCurrentProcess()->AppendSwitch(
@@ -606,33 +605,8 @@ TEST_F(PersonalDataManagerTest, UpdateUnverifiedProfilesAndCreditCards) {
EXPECT_EQ(credit_card.origin(), cards3[0]->origin());
}
-// Tests that server cards are ignored without the flag.
-TEST_F(PersonalDataManagerTest, ReturnsServerCreditCards) {
- std::vector<CreditCard> server_cards;
- server_cards.push_back(CreditCard(CreditCard::MASKED_SERVER_CARD, "a123"));
- test::SetCreditCardInfo(&server_cards.back(), "John Dillinger",
- "9012" /* Visa */, "01", "2010");
- server_cards.back().SetTypeForMaskedCard(kVisaCard);
-
- server_cards.push_back(CreditCard(CreditCard::MASKED_SERVER_CARD, "b456"));
- test::SetCreditCardInfo(&server_cards.back(), "Bonnie Parker",
- "2109" /* Mastercard */, "12", "2012");
- server_cards.back().SetTypeForMaskedCard(kMasterCard);
-
- test::SetServerCreditCards(autofill_table_, server_cards);
- personal_data_->Refresh();
-
- EXPECT_CALL(personal_data_observer_, OnPersonalDataChanged())
- .WillOnce(QuitMainMessageLoop());
- base::MessageLoop::current()->Run();
-
- EXPECT_EQ(0U, personal_data_->GetCreditCards().size());
-}
-
// Makes sure that full cards are re-masked when full PAN storage is off.
TEST_F(PersonalDataManagerTest, RefuseToStoreFullCard) {
- prefs_->SetBoolean(prefs::kAutofillWalletSyncExperimentEnabled, true);
-
// On Linux this should be disabled automatically. Elsewhere, only if the
// flag is passed.
#if defined(OS_LINUX) && !defined(OS_CHROMEOS)
« no previous file with comments | « components/autofill/core/browser/personal_data_manager.cc ('k') | components/autofill/core/common/autofill_pref_names.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698