Combine the WM_CHAR with WM_KEY* for key event flow.

ui/base/ime/remote_input_method_win.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/remote_input_method_win.h"
 
 #include "base/command_line.h"
 #include "base/observer_list.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/metro.h"
@@ skipping 165 matching lines @@
   }
 
   TextInputClient* GetTextInputClient() const override {
     return text_input_client_;
   }
 
   void DispatchKeyEvent(ui::KeyEvent* event) override {
     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;
     }
 
     if (event->is_char()) {
       if (text_input_client_) {
         text_input_client_->InsertChar(
             event->GetCharacter(),
             ui::GetModifiersFromKeyState());
       }
       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 {
     if (!text_input_client_ || text_input_client_ != client)
       return;
     std::vector<int32> prev_input_scopes;
     std::swap(input_scopes_, prev_input_scopes);
     input_scopes_ = GetInputScopesAsInt(client->GetTextInputType(),
                                         client->GetTextInputMode());
     if (input_scopes_ != prev_input_scopes && remote_delegate_) {
@@ skipping 122 matching lines @@
 
   internal::InputMethodDelegate* delegate_;
   internal::RemoteInputMethodDelegateWin* remote_delegate_;
 
   TextInputClient* text_input_client_;
   std::vector<int32> input_scopes_;
   std::vector<gfx::Rect> composition_character_bounds_;
   bool is_candidate_popup_open_;
   bool is_ime_;
   LANGID langid_;
+  std::vector<base::NativeEvent> pending_char_events_;
 
   DISALLOW_COPY_AND_ASSIGN(RemoteInputMethodWin);
 };
 
 }  // namespace
 
 bool IsRemoteInputMethodWinRequired(gfx::AcceleratedWidget widget) {
   // If the remote input method is already registered then don't do it again.
   if (ui::g_public_interface_ && ui::g_private_interface_)
     return false;
@@ skipping 17 matching lines @@
   return make_scoped_ptr(new RemoteInputMethodWin(delegate));
 }
 
 // static
 RemoteInputMethodPrivateWin* RemoteInputMethodPrivateWin::Get(
     InputMethod* input_method) {
   return GetPrivate(input_method);
 }
 
 }  // namespace ui