Merge 32166 - A quick fix for Issue 3798....

chrome/browser/autocomplete/autocomplete_edit_view_win.cc

 // Copyright (c) 2009 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 "chrome/browser/autocomplete/autocomplete_edit_view_win.h"
 
 #include <locale>
 
 #include "app/clipboard/clipboard.h"
 #include "app/clipboard/scoped_clipboard_writer.h"
@@ skipping 1234 matching lines @@
     // modifying the result set that we're in the midst of using.  For example,
     // if SetWindowTextAndCaretPos() was called due to the user clicking an
     // entry in the popup, we're in the middle of executing SetSelectedLine(),
     // and changing the results can cause checkfailures.
     return DefWindowProc(message, wparam, lparam);
   }
 
   ScopedFreeze freeze(this, GetTextObjectModel());
   OnBeforePossibleChange();
   LRESULT result = DefWindowProc(message, wparam, lparam);
+
+  // Some IMEs insert whitespace characters instead of input characters while
+  // they are composing text, and trimming these whitespace characters at the
+  // beginning of this control (in OnAfterPossibleChange()) prevents users from
+  // inputting text on these IMEs.
+  // To prevent this problem, we should not start auto-complete if the
+  // composition string starts with whitespace characters.
+  // (When we type a space key to insert a whitespace character, IMEs don't
+  // insert the whitespace character to their composition string but their
+  // result string. So, this code doesn't prevent us from updating autocomplete
+  // when we insert a whitespace character.)
+  if (lparam & GCS_COMPSTR) {
+    std::wstring text;
+    HIMC context = ImmGetContext(m_hWnd);
+    if (context) {
+      int size = ImmGetCompositionString(context, GCS_COMPSTR, NULL, 0);
+      if (size > 0) {
+        wchar_t* text_data = WriteInto(&text, size / sizeof(wchar_t) + 1);
+        if (text_data)
+          ImmGetCompositionString(context, GCS_COMPSTR, text_data, size);
+      }
+      ImmReleaseContext(m_hWnd, context);
+    }
+    if (!text.empty() && IsWhitespace(text[0]))
+      return result;
+  }
+
   if (!OnAfterPossibleChange() && (lparam & GCS_RESULTSTR)) {
     // The result string changed, but the text in the popup didn't actually
     // change.  This means the user finalized the composition.  Rerun
     // autocomplete so that we can now trigger inline autocomplete if
     // applicable.
     //
     // Note that if we're in the midst of losing focus, UpdatePopup() won't
     // actually rerun autocomplete, but will just set local state correctly.
     UpdatePopup();
   }
@@ skipping 1139 matching lines @@
     context_menu_contents_->AddItemWithStringId(IDS_PASTE_AND_GO,
                                                 IDS_PASTE_AND_GO);
     context_menu_contents_->AddSeparator();
     context_menu_contents_->AddItemWithStringId(IDS_SELECT_ALL, IDS_SELECT_ALL);
     context_menu_contents_->AddSeparator();
     context_menu_contents_->AddItemWithStringId(IDS_EDIT_SEARCH_ENGINES,
                                                 IDS_EDIT_SEARCH_ENGINES);
   }
   context_menu_.reset(new views::Menu2(context_menu_contents_.get()));
 }