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

Unified Diff: chrome/browser/autofill/select_control_handler_unittest.cc

Issue 6673079: Reduce boxing and unboxing of AutofillFieldType (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 9 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: chrome/browser/autofill/select_control_handler_unittest.cc
diff --git a/chrome/browser/autofill/select_control_handler_unittest.cc b/chrome/browser/autofill/select_control_handler_unittest.cc
index 4e65ede581c1a94d40de673d2ffce8d21f891f07..16bb5931983373df2931f60186045bdab1802d4a 100644
--- a/chrome/browser/autofill/select_control_handler_unittest.cc
+++ b/chrome/browser/autofill/select_control_handler_unittest.cc
@@ -23,10 +23,8 @@ TEST(SelectControlHandlerTest, CreditCardMonthExact) {
field.option_strings = options;
CreditCard credit_card;
- credit_card.SetInfo(AutofillType(CREDIT_CARD_EXP_MONTH), ASCIIToUTF16("01"));
- autofill::FillSelectControl(credit_card,
- AutofillType(CREDIT_CARD_EXP_MONTH),
- &field);
+ credit_card.SetInfo(CREDIT_CARD_EXP_MONTH, ASCIIToUTF16("01"));
+ autofill::FillSelectControl(credit_card, CREDIT_CARD_EXP_MONTH, &field);
EXPECT_EQ(ASCIIToUTF16("01"), field.value);
}
@@ -44,10 +42,8 @@ TEST(SelectControlHandlerTest, CreditCardMonthAbbreviated) {
field.option_strings = options;
CreditCard credit_card;
- credit_card.SetInfo(AutofillType(CREDIT_CARD_EXP_MONTH), ASCIIToUTF16("01"));
- autofill::FillSelectControl(credit_card,
- AutofillType(CREDIT_CARD_EXP_MONTH),
- &field);
+ credit_card.SetInfo(CREDIT_CARD_EXP_MONTH, ASCIIToUTF16("01"));
+ autofill::FillSelectControl(credit_card, CREDIT_CARD_EXP_MONTH, &field);
EXPECT_EQ(ASCIIToUTF16("Jan"), field.value);
}
@@ -65,10 +61,8 @@ TEST(SelectControlHandlerTest, CreditCardMonthFull) {
field.option_strings = options;
CreditCard credit_card;
- credit_card.SetInfo(AutofillType(CREDIT_CARD_EXP_MONTH), ASCIIToUTF16("01"));
- autofill::FillSelectControl(credit_card,
- AutofillType(CREDIT_CARD_EXP_MONTH),
- &field);
+ credit_card.SetInfo(CREDIT_CARD_EXP_MONTH, ASCIIToUTF16("01"));
+ autofill::FillSelectControl(credit_card, CREDIT_CARD_EXP_MONTH, &field);
EXPECT_EQ(ASCIIToUTF16("January"), field.value);
}
@@ -85,10 +79,8 @@ TEST(SelectControlHandlerTest, CreditCardMonthNumeric) {
field.option_strings = options;
CreditCard credit_card;
- credit_card.SetInfo(AutofillType(CREDIT_CARD_EXP_MONTH), ASCIIToUTF16("01"));
- autofill::FillSelectControl(credit_card,
- AutofillType(CREDIT_CARD_EXP_MONTH),
- &field);
+ credit_card.SetInfo(CREDIT_CARD_EXP_MONTH, ASCIIToUTF16("01"));
+ autofill::FillSelectControl(credit_card, CREDIT_CARD_EXP_MONTH, &field);
EXPECT_EQ(ASCIIToUTF16("1"), field.value);
}
@@ -105,10 +97,8 @@ TEST(SelectControlHandlerTest, AddressCountryFull) {
field.option_strings = options;
AutofillProfile profile;
- profile.SetInfo(AutofillType(ADDRESS_HOME_COUNTRY), ASCIIToUTF16("CA"));
- autofill::FillSelectControl(profile,
- AutofillType(ADDRESS_HOME_COUNTRY),
- &field);
+ profile.SetInfo(ADDRESS_HOME_COUNTRY, ASCIIToUTF16("CA"));
+ autofill::FillSelectControl(profile, ADDRESS_HOME_COUNTRY, &field);
EXPECT_EQ(ASCIIToUTF16("Canada"), field.value);
}
@@ -125,10 +115,8 @@ TEST(SelectControlHandlerTest, AddressCountryAbbrev) {
field.option_strings = options;
AutofillProfile profile;
- profile.SetInfo(AutofillType(ADDRESS_HOME_COUNTRY), ASCIIToUTF16("Canada"));
- autofill::FillSelectControl(profile,
- AutofillType(ADDRESS_HOME_COUNTRY),
- &field);
+ profile.SetInfo(ADDRESS_HOME_COUNTRY, ASCIIToUTF16("Canada"));
+ autofill::FillSelectControl(profile, ADDRESS_HOME_COUNTRY, &field);
EXPECT_EQ(ASCIIToUTF16("CA"), field.value);
}
@@ -145,10 +133,8 @@ TEST(SelectControlHandlerTest, AddressStateFull) {
field.option_strings = options;
AutofillProfile profile;
- profile.SetInfo(AutofillType(ADDRESS_HOME_STATE), ASCIIToUTF16("CA"));
- autofill::FillSelectControl(profile,
- AutofillType(ADDRESS_HOME_STATE),
- &field);
+ profile.SetInfo(ADDRESS_HOME_STATE, ASCIIToUTF16("CA"));
+ autofill::FillSelectControl(profile, ADDRESS_HOME_STATE, &field);
EXPECT_EQ(ASCIIToUTF16("California"), field.value);
}
@@ -165,9 +151,7 @@ TEST(SelectControlHandlerTest, AddressStateAbbrev) {
field.option_strings = options;
AutofillProfile profile;
- profile.SetInfo(AutofillType(ADDRESS_HOME_STATE), ASCIIToUTF16("California"));
- autofill::FillSelectControl(profile,
- AutofillType(ADDRESS_HOME_STATE),
- &field);
+ profile.SetInfo(ADDRESS_HOME_STATE, ASCIIToUTF16("California"));
+ autofill::FillSelectControl(profile, ADDRESS_HOME_STATE, &field);
EXPECT_EQ(ASCIIToUTF16("CA"), field.value);
}
« no previous file with comments | « chrome/browser/autofill/select_control_handler.cc ('k') | chrome/browser/automation/testing_automation_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698