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

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

Issue 7747009: Add metrics to track Autofill "user happiness" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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/form_structure_unittest.cc
diff --git a/chrome/browser/autofill/form_structure_unittest.cc b/chrome/browser/autofill/form_structure_unittest.cc
index 487662ece3e6e8af9e2649905e6cd743953e4489..a8d7a1c24b91083a994ced440071aa2da469eef4 100644
--- a/chrome/browser/autofill/form_structure_unittest.cc
+++ b/chrome/browser/autofill/form_structure_unittest.cc
@@ -1463,7 +1463,7 @@ TEST(FormStructureTest, EncodeUploadRequest) {
ASSERT_EQ(form_structure->field_count(), possible_field_types.size());
for (size_t i = 0; i < form_structure->field_count(); ++i)
- form_structure->set_possible_types(i, possible_field_types[i]);
+ form_structure->field(i)->set_possible_types(possible_field_types[i]);
FieldTypeSet available_field_types;
available_field_types.insert(NAME_FIRST);
@@ -1523,7 +1523,7 @@ TEST(FormStructureTest, EncodeUploadRequest) {
form_structure.reset(new FormStructure(form));
ASSERT_EQ(form_structure->field_count(), possible_field_types.size());
for (size_t i = 0; i < form_structure->field_count(); ++i)
- form_structure->set_possible_types(i, possible_field_types[i]);
+ form_structure->field(i)->set_possible_types(possible_field_types[i]);
EXPECT_TRUE(form_structure->EncodeUploadRequest(available_field_types, false,
&encoded_xml));
@@ -1563,7 +1563,7 @@ TEST(FormStructureTest, EncodeUploadRequest) {
form_structure.reset(new FormStructure(form));
ASSERT_EQ(form_structure->field_count(), possible_field_types.size());
for (size_t i = 0; i < form_structure->field_count(); ++i)
- form_structure->set_possible_types(i, possible_field_types[i]);
+ form_structure->field(i)->set_possible_types(possible_field_types[i]);
EXPECT_FALSE(form_structure->EncodeUploadRequest(available_field_types, false,
&encoded_xml));
}
@@ -1594,7 +1594,7 @@ TEST(FormStructureTest, CheckDataPresence) {
FieldTypeSet unknown_type;
unknown_type.insert(UNKNOWN_TYPE);
for (size_t i = 0; i < form_structure.field_count(); ++i)
- form_structure.set_possible_types(i, unknown_type);
+ form_structure.field(i)->set_possible_types(unknown_type);
// No available types.
// datapresent should be "" == trimmmed(0x0000000000000000) ==
@@ -1878,7 +1878,7 @@ TEST(FormStructureTest, CheckMultipleTypes) {
form_structure.reset(new FormStructure(form));
for (size_t i = 0; i < form_structure->field_count(); ++i)
- form_structure->set_possible_types(i, possible_field_types[i]);
+ form_structure->field(i)->set_possible_types(possible_field_types[i]);
std::string encoded_xml;
// Now we matched both fields singularly.
@@ -1896,7 +1896,7 @@ TEST(FormStructureTest, CheckMultipleTypes) {
encoded_xml);
// Match third field as both first and last.
possible_field_types[2].insert(NAME_FIRST);
- form_structure->set_possible_types(2, possible_field_types[2]);
+ form_structure->field(2)->set_possible_types(possible_field_types[2]);
EXPECT_TRUE(form_structure->EncodeUploadRequest(available_field_types, false,
&encoded_xml));
EXPECT_EQ("<\?xml version=\"1.0\" encoding=\"UTF-8\"\?>"
@@ -1911,8 +1911,7 @@ TEST(FormStructureTest, CheckMultipleTypes) {
"</autofillupload>",
encoded_xml);
possible_field_types[3].insert(ADDRESS_HOME_LINE2);
- form_structure->set_possible_types(
- form_structure->field_count() - 1,
+ form_structure->field(field_count() - 1)->set_possible_types(
possible_field_types[form_structure->field_count() - 1]);
EXPECT_TRUE(form_structure->EncodeUploadRequest(available_field_types, false,
&encoded_xml));
@@ -1931,8 +1930,7 @@ TEST(FormStructureTest, CheckMultipleTypes) {
possible_field_types[3].clear();
possible_field_types[3].insert(ADDRESS_HOME_LINE1);
possible_field_types[3].insert(COMPANY_NAME);
- form_structure->set_possible_types(
- form_structure->field_count() - 1,
+ form_structure->field(field_count() - 1)->set_possible_types(
possible_field_types[form_structure->field_count() - 1]);
EXPECT_TRUE(form_structure->EncodeUploadRequest(available_field_types, false,
&encoded_xml));

Powered by Google App Engine
This is Rietveld 408576698