summaryrefslogtreecommitdiffstats
path: root/components/autofill/browser/personal_data_manager_unittest.cc
diff options
context:
space:
mode:
Diffstat (limited to 'components/autofill/browser/personal_data_manager_unittest.cc')
-rw-r--r--components/autofill/browser/personal_data_manager_unittest.cc21
1 files changed, 6 insertions, 15 deletions
diff --git a/components/autofill/browser/personal_data_manager_unittest.cc b/components/autofill/browser/personal_data_manager_unittest.cc
index 8e67dc0..439991c 100644
--- a/components/autofill/browser/personal_data_manager_unittest.cc
+++ b/components/autofill/browser/personal_data_manager_unittest.cc
@@ -339,10 +339,8 @@ TEST_F(PersonalDataManagerTest, UpdateUnverifiedProfilesAndCreditCards) {
ASSERT_EQ(1U, cards2.size());
EXPECT_NE(profile.origin(), profiles2[0]->origin());
EXPECT_NE(credit_card.origin(), cards2[0]->origin());
- // TODO(isherman): Verify that the origins match once they are saved and read
- // from the database. http://crbug.com/170401
- // EXPECT_EQ(original_profile.origin(), profiles2[0]->origin());
- // EXPECT_EQ(original_credit_card.origin(), cards2[0]->origin());
+ EXPECT_EQ(original_profile.origin(), profiles2[0]->origin());
+ EXPECT_EQ(original_credit_card.origin(), cards2[0]->origin());
// Try to update with data changed as well.
profile.SetRawInfo(NAME_FIRST, ASCIIToUTF16("John"));
@@ -363,10 +361,8 @@ TEST_F(PersonalDataManagerTest, UpdateUnverifiedProfilesAndCreditCards) {
ASSERT_EQ(1U, cards3.size());
EXPECT_EQ(0, profile.Compare(*profiles3[0]));
EXPECT_EQ(0, credit_card.Compare(*cards3[0]));
- // TODO(isherman): Verify that the origins match once they are saved and read
- // from the database. http://crbug.com/170401
- // EXPECT_EQ(profile.origin(), profiles3[0]->origin());
- // EXPECT_EQ(credit_card.origin(), cards3[0]->origin());
+ EXPECT_EQ(profile.origin(), profiles3[0]->origin());
+ EXPECT_EQ(credit_card.origin(), cards3[0]->origin());
}
TEST_F(PersonalDataManagerTest, AddProfilesAndCreditCards) {
@@ -1923,10 +1919,7 @@ TEST_F(PersonalDataManagerTest, AggregateSameCreditCardWithSeparators) {
// Ensure that if a verified profile already exists, aggregated profiles cannot
// modify it in any way.
-// TODO(isherman): Enable this test once origins are saved and read from the
-// database. http://crbug.com/170401
-TEST_F(PersonalDataManagerTest,
- DISABLED_AggregateExistingVerifiedProfileWithConflict) {
+TEST_F(PersonalDataManagerTest, AggregateExistingVerifiedProfileWithConflict) {
// Start with a verified profile.
AutofillProfile profile;
profile.set_origin("Chrome settings");
@@ -1986,10 +1979,8 @@ TEST_F(PersonalDataManagerTest,
// Ensure that if a verified credit card already exists, aggregated credit cards
// cannot modify it in any way.
-// TODO(isherman): Enable this test once origins are saved and read from the
-// database. http://crbug.com/170401
TEST_F(PersonalDataManagerTest,
- DISABLED_AggregateExistingVerifiedCreditCardWithConflict) {
+ AggregateExistingVerifiedCreditCardWithConflict) {
// Start with a verified credit card.
CreditCard credit_card;
credit_card.set_origin("Chrome settings");