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

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: sync Created 7 years, 8 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
« no previous file with comments | « components/autofill/browser/autofill_profile.cc ('k') | components/autofill/browser/credit_card.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/autofill/browser/autofill_profile_unittest.cc
===================================================================
--- components/autofill/browser/autofill_profile_unittest.cc (revision 192613)
+++ 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) {
@@ -731,7 +731,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);
}
@@ -751,7 +751,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("CA"), field.value);
}
@@ -771,7 +771,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);
}
@@ -791,7 +791,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);
}
@@ -813,7 +813,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);
}
@@ -835,6 +835,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);
}
« no previous file with comments | « components/autofill/browser/autofill_profile.cc ('k') | components/autofill/browser/credit_card.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698