Integrate the new input method API for Views into Chromium.

views/focus/focus_manager.cc

Index: views/focus/focus_manager.cc
diff --git a/views/focus/focus_manager.cc b/views/focus/focus_manager.cc
index 5712b2e993a44a302f077719859b84b1eb28fdd7..93731e7b5561f94f75f612ba303a115cfa6871a2 100644
--- a/views/focus/focus_manager.cc
+++ b/views/focus/focus_manager.cc
@@ -368,10 +368,6 @@ void FocusManager::RestoreFocusedView() {
           focus_change_reason_ = kReasonFocusRestore;
       }
     }
-  } else {
-    // Clearing the focus will focus the root window, so we still get key
-    // events.
-    ClearFocus();

[oshima, 2011/03/22 01:50:39]

For my education, can you explain why we're removing this?

[James Su, 2011/03/22 08:41:14]

To fix NativeTextfieldViewsTest.FocusTraversalTest and input method related
tests.

The problem is: in StoreFocusedView() we don't store NULL view, thus it makes no
sense to restore a NULL view in RestoreFocusedView().

[oshima, 2011/03/25 17:31:25]

Changing the code to fix test doesn't sounds good explanation :)
Hmm, i'm not convinced (yet) if this is okay. May be fine, but it's subtle. What
happen to the existing focused view? Won't this change the behavior?
I'm adding jay who may know why we have this now.

[James Su, 2011/03/25 20:53:59]

I'm actually not fully confident of this change. Let's wait for Jay's feedback.

   }
 }