Performance fix for Vista+ where non-IME input is reported as IME

ui/base/win/ime_input.cc

 // Copyright (c) 2012 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/win/ime_input.h"
 
+#include <atlbase.h>
+#include <atlcom.h>
+#include <msctf.h>
+
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string16.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/base/ime/composition_text.h"
 
 // "imm32.lib" is required by IMM32 APIs used in this file.
 // NOTE(hbono): To comply with a comment from Darin, I have added
@@ skipping 106 matching lines @@
 ImeInput::~ImeInput() {
 }
 
 bool ImeInput::SetInputLanguage() {
   // Retrieve the current keyboard layout from Windows and determine whether
   // or not the current input context has IMEs.
   // Also save its input language for language-specific operations required
   // while composing a text.
   HKL keyboard_layout = ::GetKeyboardLayout(0);
   input_language_id_ = reinterpret_cast<LANGID>(keyboard_layout);
-  ime_status_ = (::ImmIsIME(keyboard_layout) == TRUE);
+
+  // Check TSF Input Processor first.
+  // If the active profile is TSF INPUTPROCESSOR, this is IME.
+  CComPtr<ITfInputProcessorProfileMgr> prof_mgr;

[Yohei Yukawa, 2013/05/16 08:55:02]

We prefer to use our own scoped pointer to maingain COM object.
Use base::win::ScopedComPtr<ITfInputProcessorProfileMgr> instead.

[kochi, 2013/05/16 10:29:31]

Done.

+  TF_INPUTPROCESSORPROFILE prof;
+  if (SUCCEEDED(prof_mgr.CoCreateInstance(CLSID_TF_InputProcessorProfiles)) &&
+      SUCCEEDED(prof_mgr->GetActiveProfile(GUID_TFCAT_TIP_KEYBOARD, &prof)) &&
+      prof.hkl == NULL &&
+      prof.dwProfileType == TF_PROFILETYPE_INPUTPROCESSOR) {
+      ime_status_ = true;
+  } else {
+    // If the curent language is not using TSF, check IMM32 based IMEs.
+    // As ImmIsIME always returns non-0 value on Vista+, use ImmGetIMEFileName
+    // instead to check if this HKL has any associated IME file.
+    ime_status_ = (ImmGetIMEFileName(keyboard_layout, NULL, 0) != 0);
+  }
+
   return ime_status_;
 }
 
 void ImeInput::CreateImeWindow(HWND window_handle) {
   // When a user disables TSF (Text Service Framework) and CUAS (Cicero
   // Unaware Application Support), Chinese IMEs somehow ignore function calls
   // to ::ImmSetCandidateWindow(), i.e. they do not move their candidate
   // window to the position given as its parameters, and use the position
   // of the current system caret instead, i.e. it uses ::GetCaretPos() to
   // retrieve the position of their IME candidate window.
@@ skipping 415 matching lines @@
   keystate[VK_RCONTROL] = 0;
   keystate[VK_LCONTROL] = 0;
   for (int i = 0; i <= VK_PACKET; ++i) {
     if (keystate[i] & kKeyDownMask)
       return false;
   }
   return true;
 }
 
 }  // namespace ui