diff options
author | Yohei Yukawa <yukawa@google.com> | 2014-05-30 18:10:18 +0900 |
---|---|---|
committer | Yohei Yukawa <yukawa@google.com> | 2014-05-30 18:21:11 +0900 |
commit | 3d46bab02978df14354379c4b10d7f68c3b80771 (patch) | |
tree | f07e9141c6278818cd4696fc4ec10caceacdf3c3 /services/core/java/com/android/server/InputMethodManagerService.java | |
parent | bc132a068b47994dc3f8e3c75c963f4676c2e970 (diff) | |
download | frameworks_base-3d46bab02978df14354379c4b10d7f68c3b80771.zip frameworks_base-3d46bab02978df14354379c4b10d7f68c3b80771.tar.gz frameworks_base-3d46bab02978df14354379c4b10d7f68c3b80771.tar.bz2 |
Rebuild the enabled IMEs list when changed in Settings page
With this CL, the input method switching controller is notified
even when the list of enabled IMEs is changed by System
Settings.
Ideally we should hook every setter method in
InputMethodSettings to ensure that InputMethodSettings and
InputMethodSubtypeSwitchingController can share the same list.
Until then, most of user visible weirdness should disappear
with this CL.
BUG: 15299968
Change-Id: Ie41da226d71b58d2e12057f85e4793b35e8b9f23
Diffstat (limited to 'services/core/java/com/android/server/InputMethodManagerService.java')
-rw-r--r-- | services/core/java/com/android/server/InputMethodManagerService.java | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/InputMethodManagerService.java b/services/core/java/com/android/server/InputMethodManagerService.java index fb69c86..3a8f767 100644 --- a/services/core/java/com/android/server/InputMethodManagerService.java +++ b/services/core/java/com/android/server/InputMethodManagerService.java @@ -1681,6 +1681,11 @@ public class InputMethodManagerService extends IInputMethodManager.Stub mCurMethodId = null; unbindCurrentMethodLocked(true, false); } + // Here is not the perfect place to reset the switching controller. Ideally + // mSwitchingController and mSettings should be able to share the same state. + // TODO: Make sure that mSwitchingController and mSettings are sharing the + // the same enabled IMEs list. + mSwitchingController.resetCircularListLocked(mContext); } /* package */ void setInputMethodLocked(String id, int subtypeId) { @@ -2650,7 +2655,10 @@ public class InputMethodManagerService extends IInputMethodManager.Stub setInputMethodEnabledLocked(defaultImiId, true); } } - + // Here is not the perfect place to reset the switching controller. Ideally + // mSwitchingController and mSettings should be able to share the same state. + // TODO: Make sure that mSwitchingController and mSettings are sharing the + // the same enabled IMEs list. mSwitchingController.resetCircularListLocked(mContext); } |