Fix bug of input.ime.onKeyEvent on Linux.

ui/base/ime/input_method_auralinux.cc

 // Copyright 2013 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/base/ime/input_method_auralinux.h"
 
 #include "base/auto_reset.h"
 #include "base/environment.h"
 #include "ui/base/ime/ime_bridge.h"
 #include "ui/base/ime/ime_engine_handler_interface.h"
@@ skipping 86 matching lines @@
   // If there's an active IME extension is listening to the key event, and the
   // current text input client is not password input client, the key event
   // should be dispatched to the extension engine in the two conditions:
   // 1) |filtered| == false: the ET_KEY_PRESSED event of non-character key,
   // or the ET_KEY_RELEASED event of all key.
   // 2) |filtered| == true && NeedInsertChar(): the ET_KEY_PRESSED event of
   // character key.
   if (text_input_type_ != TEXT_INPUT_TYPE_PASSWORD &&
       GetEngine() && GetEngine()->IsInterestedInKeyEvent() &&
       (!filtered || NeedInsertChar())) {
-    ui::IMEEngineHandlerInterface::KeyEventDoneCallback callback =
-        base::Bind(&InputMethodAuraLinux::ProcessKeyEventDone,
-                   weak_ptr_factory_.GetWeakPtr(),
-                   base::Owned(new ui::KeyEvent(*event)), filtered);
+    ui::IMEEngineHandlerInterface::KeyEventDoneCallback callback = base::Bind(
+        &InputMethodAuraLinux::ProcessKeyEventByEngineDone,
+        weak_ptr_factory_.GetWeakPtr(), base::Owned(new ui::KeyEvent(*event)),
+        filtered, composition_changed_, composition_, result_text_);

[Shu Chen, 2016/03/17 06:37:08]

I think this is wrong.
Please consider why the KeyEvent needs to be a pointer with base::Owned. You may
need to do the same for the |composition_| & |result_text_|.

[Azure Wei, 2016/03/17 07:02:37]

Done.

     GetEngine()->ProcessKeyEvent(*event, callback);
   } else {
     ProcessKeyEventDone(event, filtered, false);
   }
 }
 
+void InputMethodAuraLinux::ProcessKeyEventByEngineDone(
+    ui::KeyEvent* event,
+    bool filtered,
+    bool composition_changed,
+    const ui::CompositionText& composition,
+    const base::string16& result_text,
+    bool is_handled) {
+  composition_changed_ = composition_changed;
+  composition_.CopyFrom(composition);
+  result_text_ = result_text;
+  ProcessKeyEventDone(event, filtered, is_handled);
+}
+
 void InputMethodAuraLinux::ProcessKeyEventDone(ui::KeyEvent* event,
                                                bool filtered,
                                                bool is_handled) {
   DCHECK(event);
   if (is_handled)
     return;
 
   // If the IME extension has not handled the key event, passes the keyevent
   // back to the previous processing flow. Preconditions for this situation:
   // 1) |filtered| == false
@@ skipping 305 matching lines @@
     client->ConfirmCompositionText();
 
     if (GetEngine())
       GetEngine()->Reset();
   }
 
   ResetContext();
 }
 
 }  // namespace ui