DescriptionRevert 219673 "Fix Views Combobox and Tree View text input."
Reverting because of most probable regression (https://code.google.com/p/chromium/issues/detail?id=280605).
Note: We need to implement gettextinputclient in the webview class, which needs to get client from web contents.
> Fix Views Combobox and Tree View text input.
>
> Make InputMethodBridge not a TextInputClient, use actual clients.
> InputMethodBridge was simply forwarding to actual client views.
> (e.g. NativeTextfieldViews and PrefixSelector for TreeView/Combobox)
>
> Remove Views::InputMethod::On[Focus|Blur]() and their calls.
> Simply update the host's TextInputClient on View focus changes.
> Fixes r216992's regression (input not sent to Combobox/TreeView).
> The IME mode indicator also now shows in dialogs more often :)
>
> Ignore Widget activation in InputMethodBase::GetTextInputClient.
> (fixes linux_chromeos initial omnibox input with this patch, seems ok)
>
> Call View::On[Focus|Blur] from Combobox and TreeView.
> (mostly for correctness and NotifyAccessibilityEvent, not critical)
> Remove the OnCaretBoundsChanged call from TreeView.
> (unnecessary and doesn't set correct candidate window bounds)
> Remove unused DesktopRootWindowHost::OnNativeWidget[Focus|Blur].
> Handle NULL TextInputClient* objects in TextInputTestHelper.
>
> BUG=276720, 271754, 265337
> TEST=Test Win (desktop, Win8 Metro, and Aura builds), CrOS, and Linux Aura's Omnibox, Find Bar, Bookmark Bubble, and Edit Bookmark Dialog's textfield text entry (and text input handling to select combobox and tree view elements) with EN, JA, and other IMEs, and related IME usage (Win+Space to change IME on Win8, Alt+` to change modes, etc.). Also test with --enable-text-services-framework. Note that IME candidate windows may not appear or may appear at incorrect positions for tree views and comboboxes, but this seems like a separate issue.
> R=sky@chromium.org
>
> Review URL: https://chromiumcodereview.appspot.com/23245012
TBR=msw@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220121
Patch Set 1 #Messages
Total messages: 3 (0 generated)
|