Fix candidate window position issue.

ui/base/ime/input_method_ibus.cc

Index: ui/base/ime/input_method_ibus.cc
diff --git a/ui/base/ime/input_method_ibus.cc b/ui/base/ime/input_method_ibus.cc
index d9a35cbb629c76f7ab88d1edd5df083f75f54943..41e3a0f77b3e70f4fc0286f5e054947710fc6c0e 100644
--- a/ui/base/ime/input_method_ibus.cc
+++ b/ui/base/ime/input_method_ibus.cc
@@ -334,9 +334,16 @@ void InputMethodIBus::OnCaretBoundsChanged(const TextInputClient* client) {
   DCHECK(!IsTextInputTypeNone());
   const gfx::Rect rect = GetTextInputClient()->GetCaretBounds();
 
+  gfx::Rect composition_head;
+  GetTextInputClient()->GetCompositionCharacterBounds(0, &composition_head);

[Yusuke Sato, 2012/06/13 14:31:26]

I guess when it returns false, composition_head is untouched, correct? If this
is correct, is the behavior well-documented?

[Seigo Nonaka, 2012/06/13 15:07:18]

It is correct but I didn't documented well about the behavior. I add comment for
TextInputClient. Also I add clear code just in case.
On 2012/06/13 14:31:26, Yusuke Sato wrote:

+
   // This function runs asynchronously.
-  ibus_client_->SetCursorLocation(
-      context_, rect.x(), rect.y(), rect.width(), rect.height());
+  ibus_client_->SetCursorLocation(context_,
+                                  rect.x(),
+                                  rect.y(),
+                                  rect.width(),
+                                  rect.height(),
+                                  composition_head);
 }
 
 void InputMethodIBus::CancelComposition(const TextInputClient* client) {