IME (input method editor) support for Aura, part 3 of 3: Use ui::InputMethod in ash.

ui/base/ime/text_input_client.h

 // Copyright (c) 2011 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.
 
 #ifndef UI_BASE_IME_TEXT_INPUT_CLIENT_H_
 #define UI_BASE_IME_TEXT_INPUT_CLIENT_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/i18n/rtl.h"
@@ skipping 42 matching lines @@
   // This method will be called whenever a char is generated by the keyboard,
   // even if the current text input type is TEXT_INPUT_TYPE_NONE.
   virtual void InsertChar(char16 ch, int flags) = 0;
 
   // Input context information -------------------------------------------------
 
   // Returns current text input type. It could be changed and even becomes
   // TEXT_INPUT_TYPE_NONE at runtime.
   virtual ui::TextInputType GetTextInputType() const = 0;
 
-  // Returns current caret (insertion point) bounds relative to the View’s
+  // Returns current caret (insertion point) bounds relative to the screen
   // coordinates. If there is selection, then the selection bounds will be
   // returned.
+  // TODO(yusukes): Currently views::NativeTextfieldViews which implements this
+  // interface returns its view's coordinates. We should to do the following:
+  // 1) Modify NativeTextfieldViews so it returns screen coordinates.
+  // 2) Remove view-to-screen coordinates conversion code in InputMethodBridge.
+  // 3) Modify InputMethodWin. It requires a rect in toplevel window's
+  //    coordinates instead of screen.
   virtual gfx::Rect GetCaretBounds() = 0;
 
   // Returns true if there is composition text.
   virtual bool HasCompositionText() = 0;
 
   // Document content operations ----------------------------------------------
 
   // Retrieves the UTF-16 based character range containing accessibled text in
   // the View. It must cover the composition and selection range.
   // Returns false if the information cannot be retrieved right now.
@@ skipping 34 matching lines @@
   // alignment of the current text box. It’s for supporting ctrl-shift on
   // Windows.
   // Returns false if the operation is not supported.
   virtual bool ChangeTextDirectionAndLayoutAlignment(
       base::i18n::TextDirection direction) = 0;
 };
 
 }  // namespace ui
 
 #endif  // UI_BASE_IME_TEXT_INPUT_CLIENT_H_