Add InputScope support into InputMethodIMM32

ui/base/ime/input_method_imm32.cc

 // Copyright (c) 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_imm32.h"
 
 #include "base/basictypes.h"
 #include "ui/base/ime/composition_text.h"
 #include "ui/base/ime/text_input_client.h"
+#include "ui/base/ime/win/tsf_input_scope.h"
 
 
 namespace ui {
 
 InputMethodIMM32::InputMethodIMM32(internal::InputMethodDelegate* delegate,
                                    HWND toplevel_window_handle)
     : InputMethodWin(delegate, toplevel_window_handle),
       enabled_(false), is_candidate_popup_open_(false),
       composing_window_handle_(NULL) {
   // In non-Aura environment, appropriate callbacks to OnFocus() and OnBlur()
@@ skipping 251 matching lines @@
       imm32_manager_.DisableIME(window_handle);
       enabled_ = false;
       break;
     default:
       imm32_manager_.EnableIME(window_handle);
       enabled_ = true;
       break;
   }
 
   imm32_manager_.SetTextInputMode(window_handle, GetTextInputMode());
+  tsf_inputscope::SetInputScopeForTsfUnawareWindow(
+      window_handle, GetTextInputType(), GetTextInputMode());

[yoichio, 2013/09/23 12:09:54]

Can we cache values of GetTextInputType() and *Mode() in this scope?

[yukawa, 2013/09/23 17:54:21]

I think we can in this particular case, although this is not a trivial question
in general.

We should make a snapshot of state if and only if
  A. the state is guaranteed to be immutable in the scope
or
  B. the state might be changed in the scope but we should use the initial state
intentionally.

In this case, |imm32_manager_.DisableIME|, |imm32_manager_.EnableIME|, and
|imm32_manager_.SetTextInputMode| might have side side effect.

Apparently InputMethodIMM32::UpdateIMEState is not designed to be reentrant.
Paradoxically it means that we assume the case A is implicitly applied in this
function. This is why I think we can *cache* the state in this particular case.

 }
 
 }  // namespace ui