Run event executor on the ui thread to remove the need to explicitly XFlush() the XTest calls.

remoting/host/event_executor_win.cc

 // Copyright (c) 2010 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 "remoting/host/event_executor_win.h"
 
 #include <windows.h>
 
 #include "base/message_loop.h"
 #include "base/stl_util-inl.h"
 #include "remoting/host/capturer.h"
 #include "remoting/proto/event.pb.h"
 #include "ui/base/keycodes/keyboard_codes.h"
 
 namespace remoting {
 
 using protocol::MouseEvent;
 using protocol::KeyEvent;
 
 EventExecutorWin::EventExecutorWin(
-    MessageLoop* message_loop, Capturer* capturer)
+    MessageLoopForUI* message_loop, Capturer* capturer)
     : message_loop_(message_loop),
       capturer_(capturer) {
 }
 
 EventExecutorWin::~EventExecutorWin() {
 }
 
 void EventExecutorWin::InjectKeyEvent(const KeyEvent* event, Task* done) {
   if (MessageLoop::current() != message_loop_) {
     message_loop_->PostTask(
@@ skipping 41 matching lines @@
   }
 
   // Flag to mark keyup events. Default is keydown.
   if (!down) {
     input.ki.dwFlags |= KEYEVENTF_KEYUP;
   }
 
   SendInput(1, &input, sizeof(INPUT));
 }
 
-protocol::InputStub* CreateEventExecutor(MessageLoop* message_loop,
+protocol::InputStub* CreateEventExecutor(MessageLoopForUI* message_loop,
                                          Capturer* capturer) {
   return new EventExecutorWin(message_loop, capturer);
 }
 
 void EventExecutorWin::HandleMouse(const MouseEvent* event) {
   // TODO(garykac) Collapse mouse (x,y) and button events into a single
   // input event when possible.
   if (event->has_x() && event->has_y()) {
     int x = event->x();
     int y = event->y();
@@ skipping 48 matching lines @@
     } else {
       button_event.mi.dwFlags =
           down ? MOUSEEVENTF_LEFTDOWN : MOUSEEVENTF_LEFTUP;
     }
 
     SendInput(1, &button_event, sizeof(INPUT));
   }
 }
 
 }  // namespace remoting