Fix Views Combobox and Tree View text input.

ui/views/controls/tree/tree_view.cc

Index: ui/views/controls/tree/tree_view.cc
diff --git a/ui/views/controls/tree/tree_view.cc b/ui/views/controls/tree/tree_view.cc
index 1d550a2ff75b849947c3e67e19514f9befc3c497..fdab98077ac7c50a2664d26b6bb4c5c2173045ed 100644
--- a/ui/views/controls/tree/tree_view.cc
+++ b/ui/views/controls/tree/tree_view.cc
@@ -22,7 +22,6 @@
 #include "ui/views/controls/scroll_view.h"
 #include "ui/views/controls/textfield/textfield.h"
 #include "ui/views/controls/tree/tree_view_controller.h"
-#include "ui/views/ime/input_method.h"
 
 using ui::TreeModel;
 using ui::TreeModelNode;
@@ -600,13 +599,10 @@ void TreeView::OnPaint(gfx::Canvas* canvas) {
 void TreeView::OnFocus() {
   View::OnFocus();
   SchedulePaintForNode(selected_node_);
-
-  // Notify the InputMethod so that it knows to query the TextInputClient.
-  if (GetInputMethod())
-    GetInputMethod()->OnCaretBoundsChanged(this);

[sky, 2013/08/21 23:35:29]

I'm pretty sure I found that if I didn't invoke this inputmethod didn't work.
Could you verify if it's necessary?

[msw, 2013/08/22 18:07:10]

I tested English and Japanese in Win7, Win8 Desktop, Win8 Metro, and the input
method works, but I found a tangential issue. With or without this call, Win7
misplaces the IME candidate window for the Tree View and Combobox. It seems that
this was broken before the general input regression (r216992). I recommend
landing and merging this fix to M-30 (with this unnecessary OnCaretBoundsChanged
call removed) and handling the candidate window placement issue separately.

 }
 
 void TreeView::OnBlur() {
+  View::OnBlur();
   SchedulePaintForNode(selected_node_);
   if (selector_)
     selector_->OnViewBlur();