Fixs bug resulting in double event delivery in mus

ui/views/test/views_test_base.h

Index: ui/views/test/views_test_base.h
diff --git a/ui/views/test/views_test_base.h b/ui/views/test/views_test_base.h
index ebdc44070cbe3f622a05d5f960f7dbb9c134c807..87b877fb0370b53f24dfdfdaa72c50b25186be15 100644
--- a/ui/views/test/views_test_base.h
+++ b/ui/views/test/views_test_base.h
@@ -48,6 +48,11 @@ class ViewsTestBase : public PlatformTest {
   // Simulate an OS-level destruction of the native window held by |widget|.
   void SimulateNativeDestroy(Widget* widget);
 
+  // When running in mus key events are routed to the ime. Use this to disable
+  // events being routed to ime. If you don't do this you'll need to wait for
+  // ime to ack the event.

[sadrul, 2017/03/07 03:07:19]

Are there any tests that actually does wait for the ack from ime? Or should we
just make this the default?

[sky, 2017/03/07 04:58:29]

I thought some tests might want to talk to the server, but then given we run
these tests concurrently that is likely problematic. So, changed. The casts are
a bit ick.

+  void DisableMusInputMethod(Widget* widget);
+
  protected:
   TestViewsDelegate* views_delegate() const {
     return test_helper_->views_delegate();