Makes DesktopWindowTreeHostXxx to dispatch key event to InputMethod.

ui/views/controls/textfield/textfield_unittest.cc

Index: ui/views/controls/textfield/textfield_unittest.cc
diff --git a/ui/views/controls/textfield/textfield_unittest.cc b/ui/views/controls/textfield/textfield_unittest.cc
index 2df3dd2800f2e6a10b647b5a1fa7636a6d40f830..8a81815df4b973cbcc8379ad271fc4e3971a2104 100644
--- a/ui/views/controls/textfield/textfield_unittest.cc
+++ b/ui/views/controls/textfield/textfield_unittest.cc
@@ -27,7 +27,6 @@
 #include "ui/events/event_processor.h"
 #include "ui/events/event_utils.h"
 #include "ui/events/keycodes/keyboard_codes.h"
-#include "ui/events/test/event_generator.h"
 #include "ui/gfx/render_text.h"
 #include "ui/strings/grit/ui_strings.h"
 #include "ui/views/controls/textfield/textfield_controller.h"
@@ -438,9 +437,6 @@ class TextfieldTest : public ViewsTestBase, public TextfieldController {
 #if defined(OS_MACOSX) && !defined(USE_AURA)
     fake_activation_ = test::WidgetTest::FakeWidgetIsActiveAlways();
 #endif
-
-    event_generator_.reset(
-        new ui::test::EventGenerator(GetContext(), widget_->GetNativeWindow()));
   }
 
   ui::MenuModel* GetContextMenuModel() {
@@ -476,7 +472,8 @@ class TextfieldTest : public ViewsTestBase, public TextfieldController {
                 (command ? ui::EF_COMMAND_DOWN : 0) |
                 (caps_lock ? ui::EF_CAPS_LOCK_DOWN : 0);
 
-    event_generator_->PressKey(key_code, flags);
+    ui::KeyEvent event(ui::ET_KEY_PRESSED, key_code, flags);
+    input_method_->DispatchKeyEvent(event);

[sadrul, 2015/07/19 01:07:30]

Hm. It may not be obvious for someone unfamiliar with this code that using
EventGenerator to generate key-events wouldn't do the right thing when the
event-target is a TextInputClient. Can this be made better? For example, perhaps
EventGenerator can be clever enough to know when to send the key-event through
the IME? Does that make sense? Can you please file a bug to track that work?

[Shu Chen, 2015/07/20 05:46:09]

I've reverted the change here and made EventGenerator to support dispatching key
events to input method.

[Shu Chen, 2015/07/21 03:05:52]

Sadrul, I changed this back in the latest patchset.
It is hard to make EventGenerator support input method in this cl.
I've opened crbug.com/512315 to track the effort.

   }
 
   void SendKeyEvent(ui::KeyboardCode key_code,
@@ -647,7 +644,6 @@ class TextfieldTest : public ViewsTestBase, public TextfieldController {
  private:
   ui::ClipboardType copied_to_clipboard_;
   scoped_ptr<test::WidgetTest::FakeActivation> fake_activation_;
-  scoped_ptr<ui::test::EventGenerator> event_generator_;
 
   DISALLOW_COPY_AND_ASSIGN(TextfieldTest);
 };