diff options
| author | yukishiino@chromium.org <yukishiino@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-06 05:45:14 +0000 |
|---|---|---|
| committer | yukishiino@chromium.org <yukishiino@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-06 05:45:14 +0000 |
| commit | 49cfc0f69d95f5bd8f5aa92c844c3a4719aaffab (patch) | |
| tree | 4a1bf972583f7cd4bf1c070ca0fa57c3a2143588 /ui/views | |
| parent | 29e052312959f3ad4cf91529290dd2f4bc3ddccf (diff) | |
| download | chromium_src-49cfc0f69d95f5bd8f5aa92c844c3a4719aaffab.zip chromium_src-49cfc0f69d95f5bd8f5aa92c844c3a4719aaffab.tar.gz chromium_src-49cfc0f69d95f5bd8f5aa92c844c3a4719aaffab.tar.bz2 | |
Fixes twice-IME-committement on blur.
This CL clears composition data in IME in InputMethodBridge::OnBlur.
The cause was that ConfirmComposition called in InputMethodBridge::OnBlue doesn't clear IME's composition, and it will be committed again in host_->SetFocusedTextInputClient(NULL) in case of IMM32.
BUG=265928
TEST=Test manually.
Review URL: https://chromiumcodereview.appspot.com/21874003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215803 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/views')
| -rw-r--r-- | ui/views/ime/input_method_bridge.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/ui/views/ime/input_method_bridge.cc b/ui/views/ime/input_method_bridge.cc index 3d9a458..34f6e1b 100644 --- a/ui/views/ime/input_method_bridge.cc +++ b/ui/views/ime/input_method_bridge.cc @@ -45,7 +45,10 @@ void InputMethodBridge::OnFocus() { } void InputMethodBridge::OnBlur() { - ConfirmCompositionText(); + if (HasCompositionText()) { + ConfirmCompositionText(); + host_->CancelComposition(this); + } if (host_->GetTextInputClient() == this) host_->SetFocusedTextInputClient(NULL); |