Refactor Textfield and AutocompleteEditViewViews.

views/controls/textfield/native_textfield_views.h

Index: views/controls/textfield/native_textfield_views.h
diff --git a/views/controls/textfield/native_textfield_views.h b/views/controls/textfield/native_textfield_views.h
index 1f801470b7a39c4473098cc344ae279fd5576539..74019cf6d0fdf5fad3bfd1bd2ec3fdf24b57cf9b 100644
--- a/views/controls/textfield/native_textfield_views.h
+++ b/views/controls/textfield/native_textfield_views.h
@@ -169,10 +169,6 @@ class NativeTextfieldViews : public views::View,
   // Handle the keyevent.
   bool HandleKeyEvent(const KeyEvent& key_event);
 
-  // Utility function. Gets the character corresponding to a keyevent.
-  // Returns 0 if the character is not printable.
-  char16 GetPrintableChar(const KeyEvent& key_event);
-
   // Find a cusor position for given |point| in this views coordinates.
   size_t FindCursorPosition(const gfx::Point& point) const;
 
@@ -194,6 +190,15 @@ class NativeTextfieldViews : public views::View,
   // Utility function to create the context menu if one does not already exist.
   void InitContextMenuIfRequired();
 
+  // Convenience method to call TextfieldController::OnBeforeUserAction();
+  void OnBeforeUserAction();
+
+  // Convenience method to call TextfieldController::OnAfterUserAction();
+  void OnAfterUserAction();
+
+  // Checks if a char is ok to be inserted into the textfield.

[oshima, 2011/03/22 23:22:23]

Can you add comment that ch is modified char?

[James Su, 2011/03/22 23:49:57]

Done.

+  static bool ShouldInsertChar(char16 ch, int flags);
+
   // The parent textfield, the owner of this object.
   Textfield* textfield_;