Refactor autofill form fields.

webkit/glue/form_field.cc

Index: webkit/glue/form_field.cc
diff --git a/webkit/glue/form_field.cc b/webkit/glue/form_field.cc
index 0de8446c6fde60f30c828785102ac68f35c603cb..cbc7db203a904fe44dbf55ecb3c570f61459e0c9 100644
--- a/webkit/glue/form_field.cc
+++ b/webkit/glue/form_field.cc
@@ -21,7 +21,8 @@ namespace webkit_glue {
 
 FormField::FormField()
     : max_length(0),
-      is_autofilled(false) {
+      is_autofilled(false),
+      is_text_input(false) {
 }
 
 // TODO(jhawkins): This constructor should probably be deprecated and the
@@ -69,6 +70,11 @@ FormField::FormField(const string16& label,
     form_control_type(form_control_type),
     max_length(max_length),
     is_autofilled(is_autofilled) {
+  const char* text_types[] = {"text", "email", "search", "tel", "url",
+      "number", "date", "datetime", "datetime-local", "month", "week",
+      "time", "range"};
+  for(size_t i = 0; i < sizeof(text_types)/sizeof(const char*); ++i)
+    is_text_input |= form_control_type == ASCIIToUTF16(text_types[i]);
 }

[honten.org, 2011/05/17 04:41:18]

It's not clean ;-(

But lots of unit tests use this constructor, and changing it at once is really
messy.

How should I do?

[dhollowa, 2011/05/17 16:47:20]

We should simply have a helper function that does this.  Probably a method in 
|form_field.cc| in an anonymous namespace.  In general it is bad practice to
store a derived value.  Unless there is an explicit reason to cache.

On 2011/05/17 04:41:18, honten wrote:

[honten.org, 2011/05/17 18:45:22]

Actually, I wanted to find the function in |form_field.cc| and
|form_manager.cc|, but I cannot find it.

It might be the following function is the candidate in |form_manager.cc|, 
bool IsTextInput(const WebInputElement* element) {
  if (!element)
    return false;

  return element->isTextField() && !element->isPasswordField();
}

But as you see here, the value is derived from isTextField().

I want to drag the flag, but as you know, it is not good idea:-(

Do you have any idea?

Thanks,
On 2011/05/17 16:47:20, dhollowa wrote:

 
 FormField::~FormField() {
@@ -80,7 +86,8 @@ bool FormField::operator==(const FormField& field) const {
   return (label == field.label &&
           name == field.name &&
           form_control_type == field.form_control_type &&
-          max_length == field.max_length);
+          max_length == field.max_length &&
+          is_text_input == field.is_text_input);
 }
 
 bool FormField::operator!=(const FormField& field) const {
@@ -92,7 +99,8 @@ bool FormField::StrictlyEqualsHack(const FormField& field) const {
           name == field.name &&
           value == field.value &&
           form_control_type == field.form_control_type &&
-          max_length == field.max_length);
+          max_length == field.max_length &&
+          is_text_input == field.is_text_input);
 }
 
 std::ostream& operator<<(std::ostream& os, const FormField& field) {