Make DispatchKeyEventPostIME() asynchronous.

ui/aura/mus/input_method_mus.cc

 // Copyright (c) 2015 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 "ui/aura/mus/input_method_mus.h"
 
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "services/ui/public/interfaces/constants.mojom.h"
@@ skipping 133 matching lines @@
 void InputMethodMus::OnDidChangeFocusedClient(
     ui::TextInputClient* focused_before,
     ui::TextInputClient* focused) {
   InputMethodBase::OnDidChangeFocusedClient(focused_before, focused);
   UpdateTextInputType();
 
   // TODO(moshayedi): crbug.com/681563. Handle when there is no focused clients.
   if (!focused)
     return;
 
-  text_input_client_ = base::MakeUnique<TextInputClientImpl>(focused);
+  text_input_client_ =
+      base::MakeUnique<TextInputClientImpl>(focused, delegate_);
 
   // We are about to close the pipe with pending callbacks. Closing the pipe
   // results in none of the callbacks being run. We have to run the callbacks
   // else mus won't process the next event immediately.
   AckPendingCallbacksUnhandled();
 
   if (ime_driver_) {
     ui::mojom::StartSessionDetailsPtr details =
         ui::mojom::StartSessionDetails::New();
     details->client = text_input_client_->CreateInterfacePtrAndBind();
@@ skipping 31 matching lines @@
 
 void InputMethodMus::ProcessKeyEventCallback(
     const ui::KeyEvent& event,
     bool handled) {
   // Remove the callback as DispatchKeyEventPostIME() may lead to calling
   // AckPendingCallbacksUnhandled(), which mutates |pending_callbacks_|.
   DCHECK(!pending_callbacks_.empty());
   std::unique_ptr<EventResultCallback> ack_callback =
       std::move(pending_callbacks_.front());
   pending_callbacks_.pop_front();
-  EventResult event_result;
-  if (!handled) {
-    // If not handled by IME, try dispatching the event to delegate to see if
-    // any client-side post-ime processing needs to be done. This includes cases
-    // like backspace, return key, etc.
-    std::unique_ptr<ui::Event> event_clone = ui::Event::Clone(event);
-    ignore_result(DispatchKeyEventPostIME(event_clone->AsKeyEvent()));
-    event_result =
-        event_clone->handled() ? EventResult::HANDLED : EventResult::UNHANDLED;
-  } else {
-    event_result = EventResult::HANDLED;
-  }
+
   // |ack_callback| can be null if the standard form of DispatchKeyEvent() is
   // called instead of the version which provides a callback. In mus+ash we
   // use the version with callback, but some unittests use the standard form.
   if (ack_callback)
-    ack_callback->Run(event_result);
+    ack_callback->Run(handled ? EventResult::HANDLED : EventResult::UNHANDLED);
 }
 
 }  // namespace aura