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

Unified Diff: components/autofill/browser/autofill_profile_unittest.cc

Issue 13488009: Remove application locale cache in autofill code. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 7 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/browser/autofill_profile_unittest.cc
===================================================================
--- components/autofill/browser/autofill_profile_unittest.cc (revision 192175)
+++ components/autofill/browser/autofill_profile_unittest.cc (working copy)
@@ -512,13 +512,13 @@
autofill_test::SetProfileInfo(b.get(), "Thomas", NULL, "Jefferson",
"declaration_guy@gmail.com", "United States Government", "Monticello",
NULL, "Charlottesville", "Virginia", "22902", NULL, NULL);
- EXPECT_TRUE(a->IsSubsetOf(*b));
+ EXPECT_TRUE(a->IsSubsetOf(*b, "en-US"));
// |b| is not a subset of |a|.
- EXPECT_FALSE(b->IsSubsetOf(*a));
+ EXPECT_FALSE(b->IsSubsetOf(*a, "en-US"));
// |a| is a subset of |a|.
- EXPECT_TRUE(a->IsSubsetOf(*a));
+ EXPECT_TRUE(a->IsSubsetOf(*a, "en-US"));
// One field in |b| is different.
a.reset(new AutofillProfile);
@@ -529,7 +529,7 @@
autofill_test::SetProfileInfo(a.get(), "Thomas", NULL, "Adams",
"declaration_guy@gmail.com", NULL, NULL, NULL, NULL, NULL, NULL, NULL,
NULL);
- EXPECT_FALSE(a->IsSubsetOf(*b));
+ EXPECT_FALSE(a->IsSubsetOf(*b, "en-US"));
}
TEST(AutofillProfileTest, AssignmentOperator) {
@@ -739,7 +739,7 @@
AutofillProfile profile;
profile.SetRawInfo(ADDRESS_HOME_COUNTRY, ASCIIToUTF16("CA"));
- profile.FillSelectControl(ADDRESS_HOME_COUNTRY, &field);
+ profile.FillSelectControl(ADDRESS_HOME_COUNTRY, "en-US", &field);
EXPECT_EQ(ASCIIToUTF16("Canada"), field.value);
}
@@ -759,7 +759,7 @@
AutofillProfile profile;
profile.SetInfo(ADDRESS_HOME_COUNTRY, ASCIIToUTF16("Canada"), "en-US");
- profile.FillSelectControl(ADDRESS_HOME_COUNTRY, &field);
+ profile.FillSelectControl(ADDRESS_HOME_COUNTRY, "en-US", &field);
EXPECT_EQ(ASCIIToUTF16("CA"), field.value);
}
@@ -779,7 +779,7 @@
AutofillProfile profile;
profile.SetRawInfo(ADDRESS_HOME_STATE, ASCIIToUTF16("CA"));
- profile.FillSelectControl(ADDRESS_HOME_STATE, &field);
+ profile.FillSelectControl(ADDRESS_HOME_STATE, "en-US", &field);
EXPECT_EQ(ASCIIToUTF16("California"), field.value);
}
@@ -799,7 +799,7 @@
AutofillProfile profile;
profile.SetRawInfo(ADDRESS_HOME_STATE, ASCIIToUTF16("California"));
- profile.FillSelectControl(ADDRESS_HOME_STATE, &field);
+ profile.FillSelectControl(ADDRESS_HOME_STATE, "en-US", &field);
EXPECT_EQ(ASCIIToUTF16("CA"), field.value);
}
@@ -821,7 +821,7 @@
AutofillProfile profile;
profile.SetRawInfo(ADDRESS_HOME_STATE, ASCIIToUTF16("California"));
- profile.FillSelectControl(ADDRESS_HOME_STATE, &field);
+ profile.FillSelectControl(ADDRESS_HOME_STATE, "en-US", &field);
EXPECT_EQ(ASCIIToUTF16("California"), field.value);
}
@@ -843,6 +843,6 @@
AutofillProfile profile;
profile.SetRawInfo(ADDRESS_HOME_STATE, ASCIIToUTF16("California"));
- profile.FillSelectControl(ADDRESS_HOME_STATE, &field);
+ profile.FillSelectControl(ADDRESS_HOME_STATE, "en-US", &field);
EXPECT_EQ(ASCIIToUTF16("2"), field.value);
}

Powered by Google App Engine
This is Rietveld 408576698