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

remoting/host/simple_host_process.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.
 //
 // This is an application of a minimal host process in a Chromoting
 // system. It serves the purpose of gluing different pieces together
 // to make a functional host process for testing.
 //
 // It peforms the following functionality:
 // 1. Connect to the GTalk network and register the machine as a host.
@@ skipping 18 matching lines @@
 #include "base/threading/thread.h"
 #include "media/base/media.h"
 #include "remoting/base/tracer.h"
 #include "remoting/host/capturer_fake.h"
 #include "remoting/host/chromoting_host.h"
 #include "remoting/host/chromoting_host_context.h"
 #include "remoting/host/event_executor.h"
 #include "remoting/host/json_host_config.h"
 #include "remoting/proto/video.pb.h"
 
+#if defined(TOOLKIT_USES_GTK)
+#include "ui/gfx/gtk_util.h"
+#endif
+
 using remoting::ChromotingHost;
 using remoting::protocol::CandidateSessionConfig;
 using remoting::protocol::ChannelConfig;
 using std::string;
 using std::wstring;
 
 #if defined(OS_WIN)
 const wchar_t kDefaultConfigPath[] = L".ChromotingConfig.json";
 const wchar_t kHomeDrive[] = L"HOMEDRIVE";
 const wchar_t kHomePath[] = L"HOMEPATH";
@@ skipping 22 matching lines @@
   // Needed for the Mac, so we don't leak objects when threads are created.
   base::mac::ScopedNSAutoreleasePool pool;
 
   CommandLine::Init(argc, argv);
   const CommandLine* cmd_line = CommandLine::ForCurrentProcess();
 
   base::AtExitManager exit_manager;
   base::EnsureNSPRInit();
 
   // Allocate a chromoting context and starts it.
-  remoting::ChromotingHostContext context;
+#if defined(TOOLKIT_USES_GTK)
+  gfx::GtkInitFromCommandLine(*cmd_line);
+#endif
+  MessageLoopForUI message_loop;
+  remoting::ChromotingHostContext context(&message_loop);
   context.Start();
 
 
 #if defined(OS_WIN)
   wstring home_path = GetEnvironmentVar(kHomeDrive);
   home_path += GetEnvironmentVar(kHomePath);
 #else
   string home_path = GetEnvironmentVar(base::env_vars::kHome);
 #endif
   FilePath config_path(home_path);
@@ skipping 21 matching lines @@
       << "Cannot load media library";
 
   // Construct a chromoting host.
   scoped_refptr<ChromotingHost> host;
 
   bool fake = cmd_line->HasSwitch(kFakeSwitchName);
   if (fake) {
     remoting::Capturer* capturer =
         new remoting::CapturerFake(context.main_message_loop());
     remoting::protocol::InputStub* input_stub =
-        CreateEventExecutor(context.main_message_loop(), capturer);
+        CreateEventExecutor(context.ui_message_loop(), capturer);
     host = ChromotingHost::Create(
         &context, config, capturer, input_stub);
   } else {
     host = ChromotingHost::Create(&context, config);
   }
 
   if (cmd_line->HasSwitch(kVideoSwitchName)) {
     string video_codec = cmd_line->GetSwitchValueASCII(kVideoSwitchName);
     scoped_ptr<CandidateSessionConfig> config(
         CandidateSessionConfig::CreateDefault());
@@ skipping 14 matching lines @@
       LOG(ERROR) << "Unknown video codec: " << video_codec;
       context.Stop();
       return 1;
     }
     config->mutable_video_configs()->push_back(ChannelConfig(
         transport, remoting::protocol::kDefaultStreamVersion, codec));
     host->set_protocol_config(config.release());
   }
 
   // Let the chromoting host run until the shutdown task is executed.
-  MessageLoop message_loop(MessageLoop::TYPE_UI);
   host->Start(NewRunnableFunction(&ShutdownTask, &message_loop));
-  message_loop.Run();
+  message_loop.MessageLoop::Run();
 
   // And then stop the chromoting context.
   context.Stop();
   file_io_thread.Stop();
   return 0;
 }