Combine the WM_CHAR with WM_KEY* for key event flow.

ui/base/ime/remote_input_method_win.cc

Index: ui/base/ime/remote_input_method_win.cc
diff --git a/ui/base/ime/remote_input_method_win.cc b/ui/base/ime/remote_input_method_win.cc
index 1a589bb0668f6b235b24f755093c2812abd512fb..745e885a4b3b427f28ada9bc8727b3cc4f098b43 100644
--- a/ui/base/ime/remote_input_method_win.cc
+++ b/ui/base/ime/remote_input_method_win.cc
@@ -183,9 +183,7 @@ class RemoteInputMethodWin : public InputMethod,
     if (event->HasNativeEvent()) {
       const base::NativeEvent& native_key_event = event->native_event();
       if (native_key_event.message == WM_CHAR && text_input_client_) {
-        text_input_client_->InsertChar(
-            static_cast<base::char16>(native_key_event.wParam),
-            ui::GetModifiersFromKeyState());
+        pending_char_events_.push_back(native_key_event);
         event->StopPropagation();
       }
       return;
@@ -200,8 +198,19 @@ class RemoteInputMethodWin : public InputMethod,
       event->StopPropagation();
       return;
     }
-    if (delegate_)
+    if (delegate_) {
       ignore_result(delegate_->DispatchKeyEventPostIME(event));
+      if (text_input_client_ && pending_char_events_.size()) {
+        for (size_t i = 0; i < pending_char_events_.size(); ++i) {
+          const base::NativeEvent& native_key_event = pending_char_events_[i];
+          text_input_client_->InsertChar(
+              static_cast<base::char16>(pending_char_events_[i].wParam),
+              event->flags());
+        }
+        pending_char_events_.clear();

[James Su, 2015/08/04 17:06:44]

this cache should always be cleared for WM_KEYDOWN/WM_KEYUP

[Shu Chen, 2015/08/05 01:36:45]

Done.

+        event->StopPropagation();
+      }
+    }
   }
 
   void OnTextInputTypeChanged(const TextInputClient* client) override {
@@ -344,6 +353,7 @@ class RemoteInputMethodWin : public InputMethod,
   bool is_candidate_popup_open_;
   bool is_ime_;
   LANGID langid_;
+  std::vector<base::NativeEvent> pending_char_events_;
 
   DISALLOW_COPY_AND_ASSIGN(RemoteInputMethodWin);
 };