Add autofill preview support for Textarea

components/autofill/content/renderer/form_autofill_util.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/autofill/content/renderer/form_autofill_util.h"
 
 #include <map>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 554 matching lines @@
 
 // Sets the |field|'s "suggested" (non JS visible) value to the value in |data|.
 // Also sets the "autofilled" attribute, causing the background to be yellow.
 void PreviewFormField(const FormFieldData& data,
                       bool is_initiating_node,
                       blink::WebFormControlElement* field) {
   // Nothing to preview.
   if (data.value.empty())
     return;
 
-  // Only preview input fields. Excludes checkboxes and radio buttons, as there
-  // is no provision for setSuggestedCheckedValue in WebInputElement.
+  // Preview input and textarea fields. For input fields, excludes checkboxes
+  // and radio buttons, as there is no provision for setSuggestedCheckedValue
+  // in WebInputElement.
   WebInputElement* input_element = toWebInputElement(field);
-  if (!IsTextInput(input_element))
+  if (IsTextInput(input_element)) {
+    // If the maxlength attribute contains a negative value, maxLength()
+    // returns the default maxlength value.
+    input_element->setSuggestedValue(
+      data.value.substr(0, input_element->maxLength()));
+      input_element->setAutofilled(true);
+    if (is_initiating_node) {
+      // Select the part of the text that the user didn't type.
+      input_element->setSelectionRange(input_element->value().length(),
+                                      input_element->suggestedValue().length());
+    }

[Ilya Sherman, 2013/12/29 03:36:02]

Can this be shared across the two branches?

[ziran.sun, 2014/01/02 15:22:54]

Do you mean merge "if(IsTextInput(input_element))" & "else if
(IsTextAreaElement(*field))"? 

[Ilya Sherman, 2014/01/07 01:09:38]

It seems like the "if (is_initiating_node)" code ought to be relevant for both
branches.

[ziran.sun, 2014/01/07 16:40:41]

I am not sure if we need to do "setSelectionRange" for Textarea as well. check
"FillFormField()", it doesn't seem this is called for Textarea. Do let me know
if I missed any points though. 

+  } else if (IsTextAreaElement(*field)) {
+    WebTextAreaElement textarea = field->to<WebTextAreaElement>();
+    textarea.setSuggestedValue(data.value);
+    field->setAutofilled(true);
+  } else if (IsSelectElement(*field)) {
+    //Handle selectElement here
+  } else {
     return;
-
-  // If the maxlength attribute contains a negative value, maxLength()
-  // returns the default maxlength value.
-  input_element->setSuggestedValue(
-      data.value.substr(0, input_element->maxLength()));
-  input_element->setAutofilled(true);
-  if (is_initiating_node) {
-    // Select the part of the text that the user didn't type.
-    input_element->setSelectionRange(input_element->value().length(),
-                                     input_element->suggestedValue().length());
   }

[Ilya Sherman, 2013/12/29 03:36:02]

nit: No need for the final "else if" nor for the "else".

[ziran.sun, 2014/01/07 16:40:41]

Done.

 }
 
 std::string RetrievalMethodToString(
     const WebElementDescriptor::RetrievalMethod& method) {
   switch (method) {
     case WebElementDescriptor::CSS_SELECTOR:
       return "CSS_SELECTOR";
     case WebElementDescriptor::ID:
       return "ID";
     case WebElementDescriptor::NONE:
@@ skipping 358 matching lines @@
   for (ScopedVector<FormFieldData>::const_iterator iter = form_fields.begin();
        iter != form_fields.end(); ++iter) {
     form->fields.push_back(**iter);
   }
 
   return true;
 }
 
 bool FindFormAndFieldForInputElement(const WebInputElement& element,
                                      FormData* form,
-                                     FormFieldData* field,
+                                       FormFieldData* field,

[Ilya Sherman, 2013/12/29 03:36:02]

nit: Spurious diff, please revert.

[ziran.sun, 2014/01/07 16:40:41]

Done.

                                      RequirementsMask requirements) {
   if (!IsAutofillableElement(element))
     return false;
 
   const WebFormElement form_element = element.form();
   if (form_element.isNull())
     return false;
 
   ExtractMask extract_mask =
       static_cast<ExtractMask>(EXTRACT_VALUE | EXTRACT_OPTIONS);
@@ skipping 62 matching lines @@
 bool ClearPreviewedFormWithElement(const WebInputElement& element,
                                    bool was_autofilled) {
   WebFormElement form_element = element.form();
   if (form_element.isNull())
     return false;
 
   std::vector<WebFormControlElement> control_elements;
   ExtractAutofillableElements(form_element, REQUIRE_AUTOCOMPLETE,
                               &control_elements);
   for (size_t i = 0; i < control_elements.size(); ++i) {
-    // Only text input elements can be previewed.
+    // text input and textarea elements can be previewed.

[Ilya Sherman, 2013/12/29 03:36:02]

nit: Please keep the word "only"

[ziran.sun, 2014/01/07 16:40:41]

Done.

     WebInputElement* input_element = toWebInputElement(&control_elements[i]);
-    if (!IsTextInput(input_element))
+    if (!IsTextInput(input_element) && !IsTextAreaElement(control_elements[i]))
       continue;
 
-    // If the input element is not auto-filled, we did not preview it, so there
-    // is nothing to reset.
-    if (!input_element->isAutofilled())
+    // If the control element is not auto-filled, we did not preview it,

[Ilya Sherman, 2013/12/29 03:36:02]

nit: I'd omit "control", i.e. simply "If the element is not autofilled, ..."

[ziran.sun, 2014/01/07 16:40:41]

Done.

+    // so there is nothing to reset.
+    if(!control_elements[i].isAutofilled())
       continue;
 
     // There might be unrelated elements in this form which have already been
     // auto-filled.  For example, the user might have already filled the address
     // part of a form and now be dealing with the credit card section.  We only
     // want to reset the auto-filled status for fields that were previewed.
-    if (input_element->suggestedValue().isEmpty())
+    if ((IsTextInput(input_element) &&
+       input_element->suggestedValue().isEmpty())
+       || (IsTextAreaElement(control_elements[i]) &&
+       control_elements[i].to<WebTextAreaElement>().suggestedValue().isEmpty()))

[Ilya Sherman, 2013/12/29 03:36:02]

nit: The text wrapping here doesn't match the Chromium style guide's
recommendations.  I would recommend either decomposing a helper method, or
formatting the code as something like:

   WebFormControlElement control_element = control_elements[i];
   if ((IsTextInput(input_element) &&
        input_element->suggestedValue().isEmpty()) ||
       (IsTextAreaElement(control_element) &&
        control_element.to<WebTextAreaElement>().suggestedValue().isEmpty()))
     continue;

[ziran.sun, 2014/01/07 16:40:41]

Done.

       continue;
 
     // Clear the suggested value. For the initiating node, also restore the
     // original value.
-    input_element->setSuggestedValue(WebString());
-    bool is_initiating_node = (element == *input_element);
-    if (is_initiating_node)
-      input_element->setAutofilled(was_autofilled);
-    else
-      input_element->setAutofilled(false);
+    if (IsTextInput(input_element)) {
+      input_element->setSuggestedValue(WebString());
+      bool is_initiating_node = (element == *input_element);
+      if (is_initiating_node)
+        input_element->setAutofilled(was_autofilled);
+      else
+        input_element->setAutofilled(false);

[Ilya Sherman, 2013/12/29 03:36:02]

Can this code be shared between the two branches?

 
-    // Clearing the suggested value in the focused node (above) can cause
-    // selection to be lost. We force selection range to restore the text
-    // cursor.
-    if (is_initiating_node) {
-      int length = input_element->value().length();
-      input_element->setSelectionRange(length, length);
+      // Clearing the suggested value in the focused node (above) can cause
+      // selection to be lost. We force selection range to restore the text
+      // cursor.
+      if (is_initiating_node) {
+        int length = input_element->value().length();
+        input_element->setSelectionRange(length, length);
+      }
+    } else if (IsTextAreaElement(control_elements[i])) {
+      WebTextAreaElement txtarea = control_elements[i].to<WebTextAreaElement>();

[Ilya Sherman, 2013/12/29 03:36:02]

nit: Please avoid abbreviations in names.  The way to format this line is

WebTextAreaElement text_area =
    control_elements[i].to<WebTextAreaElement>();

[ziran.sun, 2014/01/07 16:40:41]

Done.

+      txtarea.setSuggestedValue(WebString());
+      bool is_initiating_node = (element == txtarea);
+      if (is_initiating_node)
+        control_elements[i].setAutofilled(was_autofilled);
+      else
+        control_elements[i].setAutofilled(false);
     }
   }
 
   return true;
 }
 
 bool FormWithElementIsAutofilled(const WebInputElement& element) {
   WebFormElement form_element = element.form();
   if (form_element.isNull())
     return false;
@@ skipping 57 matching lines @@
         break;
       }
     }
     if (!tag_is_allowed)
       return false;
   }
   return true;
 }
 
 }  // namespace autofill