Add text field change handling for autofill preview in textarea element.

public/web/WebAutofillClient.h

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef WebAutofillClient_h
 #define WebAutofillClient_h
 
 namespace blink {
 
+class WebFormControlElement;
 class WebFormElement;
 class WebFrame;
 class WebInputElement;
 class WebKeyboardEvent;
 class WebNode;
 
 template <typename T> class WebVector;
 
 class WebAutofillClient {
 public:
     // Informs the browser an interactive autocomplete has been requested.
     virtual void didRequestAutocomplete(WebFrame*, const WebFormElement&) { }
 
     // These methods are called when the users edits a text-field.
     virtual void textFieldDidEndEditing(const WebInputElement&) { }
-    virtual void textFieldDidChange(const WebInputElement&) { }
+    virtual void textFieldDidChange(const WebFormControlElement&) { }

[tkent, 2014/03/04 00:15:56]

This will break build because Chromium has "textFieldDidChange(const
WebInputElement&) OVERRIDE".  Please *add* textFieldDidChange(const
WebFormControlElement&).

[jamesr, 2014/03/04 00:26:33]

If so then the bug is the OVERRIDE annotation on the chromium side. 
Implementations of cross-repo APIs should *never* have OVERRIDE annotations.

[ziran.sun, 2014/03/04 12:12:40]

Do you mean add "textFieldDidChange(const
WebInputElement&)" here?

[ziran.sun, 2014/03/04 12:12:40]

In the chromium side code for this bug, we removed OVERRIDE for both 
textFieldDidChange(const WebInputElement&) and
textFieldDidChange(const WebFormControlElement&)

The end target though is to replace textFieldDidChange(const WebInputElement&)
with textFieldDidChange(const WebInputElement&). Reason for keeping both for the
moment is due to the Blink roll delay.  

So once both Blink and Chromium side code are merged in, we'll remove
textFieldDidChange(const WebInputElement&). Then we put OVERRIDE on
textFieldDidChange(const WebFormControlElement&) - this is the stage I'm not
sure. If I understand correctly, your mean OVERRIDE shouldn't be used here at
all? 

     virtual void textFieldDidReceiveKeyDown(const WebInputElement&, const WebKeyboardEvent&) { }
     // This is called when a datalist indicator is clicked.
     virtual void openTextDataListChooser(const WebInputElement&) { }
 
     // Informs the client whether or not any subsequent text changes should be ignored.
     virtual void setIgnoreTextChanges(bool ignore) { }
 
     virtual void didAssociateFormControls(const WebVector<WebNode>&) { }
 
 protected:
     virtual ~WebAutofillClient() { }
 };
 
 } // namespace blink
 
 #endif