Fixs bug resulting in double event delivery in mus

ui/views/test/views_test_base.h

 // Copyright (c) 2012 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.
 
 #ifndef UI_VIEWS_TEST_VIEWS_TEST_BASE_H_
 #define UI_VIEWS_TEST_VIEWS_TEST_BASE_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 30 matching lines @@
 
   // Creates a widget of |type| with any platform specific data for use in
   // cross-platform tests.
   Widget::InitParams CreateParams(Widget::InitParams::Type type);
 
   bool HasCompositingManager() const;
 
   // 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();
   }
 
   void set_views_delegate(std::unique_ptr<TestViewsDelegate> views_delegate) {
     DCHECK(!setup_called_);
     views_delegate_for_setup_.swap(views_delegate);
   }
 
@@ skipping 14 matching lines @@
 #if defined(OS_WIN)
   ui::ScopedOleInitializer ole_initializer_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(ViewsTestBase);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_TEST_VIEWS_TEST_BASE_H_