aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin/LatinIME.java (follow)
Commit message (Collapse)AuthorAgeFilesLines
* [IL124] Remove the KeyboardSwitcher arg to onCodeInputJean Chalard2014-03-061-2/+13
| | | | | Bug: 8636060 Change-Id: I29ce4c82ea880a937b1c9e5daf494fcdbbdd1336
* [IL120] CleanupJean Chalard2014-03-041-4/+4
| | | | | | Remove useless arguments, local vars, make stuff private Change-Id: I7cc6c19e9c1703304d593e02be31eb9569cc7b4e
* [IL118] Factorize a methodJean Chalard2014-03-041-1/+1
| | | | | | | If we can't get the right locale from the subtype switcher, we should fallback in all cases. Change-Id: I46a53a67927a793d9d3592a996a5a4392a36c6ff
* [IL117] SimplificationJean Chalard2014-03-041-12/+12
| | | | Change-Id: Id4d4f30ecdc80dd59543f90fe2525f4bcc5afb9d
* [IL116] Inline a methodJean Chalard2014-03-041-15/+10
| | | | Change-Id: I6af8f2bd6b4917f9b119ff78a0f299b2f6fb921e
* [IL114] Move a method to InputLogicJean Chalard2014-03-041-22/+0
| | | | | Bug: 8636060 Change-Id: I0d8f93985ddc6e93ce9af0d92a8056620351a498
* [IL113] More cleanupJean Chalard2014-03-041-9/+5
| | | | | | | | | | | We should not test punctuation with this equality test any more. Also, whether the suggestion strip is displayed or not, or whether the hint is displayed or not, has nothing to do with this method and should be handled elsewhere rather than here -- and as a matter of fact, it is, which makes this useless. Bug: 8636060 Change-Id: I6a54ee87e4e9f81bc33158acf4a264c3abd5829d
* [IL112] CleanupJean Chalard2014-03-041-6/+1
| | | | | Bug: 8636060 Change-Id: Ic7c2dec2521613d52707457463691bec383177ff
* [IL111] Remove a member.Jean Chalard2014-03-041-6/+0
| | | | | | | ...and improve encapsulation and readability. Bug: 8636060 Change-Id: I36407630b307df0721ea7fdb7e7a22cdf2b018b3
* Merge changes I20ce61c4,I2edab4e3Keisuke Kuroyanagi2014-03-031-23/+27
|\ | | | | | | | | | | * changes: Remove "Use only personalization dictionary". Remove redundant creation of dictionary facilitator.
| * Remove redundant creation of dictionary facilitator.Keisuke Kuroyanagi2014-02-271-23/+27
| | | | | | | | Change-Id: I2edab4e3ba4ea0b9b13f9bb224aac86cac9b9f31
* | Add missing null checks for getOwnerInstance()Ken Wakasa2014-02-281-11/+33
| | | | | | | | | | | | It may just return null. Change-Id: I778fb48262ef8af28a4af50b70bb94ace75a488c
* | [IL110] Remove an unused interface functionJean Chalard2014-02-261-3/+1
| | | | | | | | | | | | | | ...and make a method private. Bug: 8636060 Change-Id: If16f7e8ef3779f77a9db23fb7c54e7c23b043b86
* | Remove useless args and processing.Jean Chalard2014-02-261-16/+10
|/ | | | | Bug: 13062385 Change-Id: I426545d1fe8fd76bcf4d1dce19e6b571a55c79ae
* Merge "Change a heuristic and add a log."Jean Chalard2014-02-211-0/+2
|\
| * Change a heuristic and add a log.Jean Chalard2014-02-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The heuristic in RichInputConnection makes little sense when textLength > mExpectedSelStart but we have more than 1024 characters of text. If there are that many, it's about 100% sure that 1024 is not the correct cursor position. With no good guess, we'll just continue trusting the app, even though we know it's lying : at least it will make the problem visible to the app author. Also, there have been a lot of confusion about initialSelStart and initialSelEnd. The keyboard should log them so that it helps us and editor authors debug more easily these common problems. Issue #65170 in AOSP and Bug: 12772035 Change-Id: I6665a16c9f2832d33ee323f033bb38bcc092a3b4
* | Add ImportantNoticeDialogTadashi G. Takaoka2014-02-211-45/+15
| | | | | | | | Change-Id: Icaeb7dddd16362cd4a6384d295f89c7fbabc8b35
* | Support versioned important noticeTadashi G. Takaoka2014-02-211-3/+16
|/ | | | | | | This change must be checked in together with If38a754d00. Bug: 10587358 Change-Id: I91580f9468bc5ee7ed1694d7852a60dc6793dac0
* Merge "Don't distrust the cursor pos so much as to bug on IPC delay"Jean Chalard2014-02-201-3/+6
|\
| * Don't distrust the cursor pos so much as to bug on IPC delayJean Chalard2014-02-201-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the cursor is moved by the user, the RichInputConnection is told about it. However, to work around a framework bug, it also looks at how many characters are in the buffer before the cursor, and if that's more than the value it's been passed, it deduces that's a framework bug and there are at least as many characters as seen before the cursor, so it puts the expected cursor position there. When you move the cursor, TextView calls onUpdateSelection, and when you move it fast, you'll get rapid-fire calls to onUpdateSelection. This is fine, the RIC is equipped to deal with that. However, these calls take some time to make it to the IME. In this instance, when the first call gets through and the IME calls TextView (synchronously) for text before the cursor, the cursor has already moved in the app, and TextView returns more characters than the cursor position was declared to be in this instance, so the RIC sets that as the expected cursor position. Sure enough, a split second later, the second call to onUpdateSelection arrives, with the new cursor position set where the RIC had found it too early. The RIC takes that as an "expected" cursor move, and the input does not get reset. Luckily, we have a way out. As far as we know, the framework bug only manifests itself upon rotation, which means we should only have to adjust for it in onStartInputView. Doing it in onUpdateSelection is too zealous (and probably too distrustful of the app to send the correct cursor positions). So we should just take care of the rotation case (by calling tryFixLyingCursorPosition in onStartInputView) and remove the compensating code in resetCachesUponCursorMoves. Bug: 12982502 Change-Id: Ic3c1408a1ec45deaea63b01d98376a79ae567d77
* | Add test only methods to test personalized dicts.Keisuke Kuroyanagi2014-02-201-2/+10
| | | | | | | | | | Bug: 11122480 Change-Id: I79a4a73ff09a2d34d4ca094d705e901fb4bc855c
* | Merge "Check all enabled IME's subtypes to enable personalization."Keisuke Kuroyanagi2014-02-201-1/+1
|\ \ | |/ |/|
| * Check all enabled IME's subtypes to enable personalization.Keisuke Kuroyanagi2014-02-201-1/+1
| | | | | | | | | | | | Bug: 13067743 Change-Id: I6fc68b5151430a5ab51777f4c381e20f5a657924
* | Change important notice version preference fileTadashi G. Takaoka2014-02-201-30/+15
|/ | | | | | | This CL also reset the notice version to 1. This CL must be checked in together with Iea8d9ff038. Change-Id: I838f7b5b295388d64d6aba90f50b06654eba16e5
* Call PersonalizationDictionarySessionRegistrar.init().Keisuke Kuroyanagi2014-02-191-0/+4
| | | | | | | Wrongly removed in I1f50cb137b6054dc9. Bug: 13067743 Change-Id: I27cd221cb7f6e403c3fc128d65d14dbf628b6f55
* Remove personalization dicts for multilingual users.Keisuke Kuroyanagi2014-02-191-9/+20
| | | | | Bug: 13067743 Change-Id: I1f50cb137b6054dc9442237586b70fc0aa2bf427
* Use AlertDialog.setOnDismissListenerTadashi G. Takaoka2014-02-191-2/+2
| | | | | | | AlertDialog.Builder.setOnDismissListener is supported from API 17. Bug: 10587358 Change-Id: Ie8ce8e283b803f7f6bb58ced446e676339f980eb
* Merge "Turn personalization on when important notice dialog is displayed"Tadashi G. Takaoka2014-02-191-19/+43
|\
| * Turn personalization on when important notice dialog is displayedTadashi G. Takaoka2014-02-191-19/+43
| | | | | | | | | | | | | | This change must be checked in together with Ifde70b1d3b. Bug: 10587358 Change-Id: I9dc48f9d521e0e6f6269e4f184389ba805f3053f
* | Fix: Use old suggest after reloading.Keisuke Kuroyanagi2014-02-191-1/+3
|/ | | | Change-Id: Icb2821f61c4203254b7d573cad5ce71fbf0ed841
* Merge "Cleanup."Jean Chalard2014-02-181-7/+1
|\
| * Cleanup.Jean Chalard2014-02-181-7/+1
| | | | | | | | | | | | | | Less code, less chance of stuff escaping, and a bit of a clarification. Yay. Change-Id: I3193fd8ad7ac28195260cbc231c4e498a63a460c
* | Some cosmetic fixes in LatinIME.javaKen Wakasa2014-02-181-40/+38
|/ | | | Change-Id: Ia2d04ab992f5a0cc736150b2691b2f8ae8f0955d
* Cosmetic fixesKen Wakasa2014-02-181-23/+30
| | | | Change-Id: I6c755565ba7689357d02d26cc0e92a93031953e8
* Set neutral suggestions should be called after load settingsTadashi G. Takaoka2014-02-181-33/+33
| | | | | Bug: 13058751 Change-Id: Iba49e86b90d595473d91753827d499e551dad45c
* Update important notice version even when user select SettingsTadashi G. Takaoka2014-02-181-1/+1
| | | | | Bug: 13060852 Change-Id: I463fdf3d51cacf99693fcde2998ea751046d1cc5
* Merge "Don't show important notice on password field"Tadashi G. Takaoka2014-02-171-4/+6
|\
| * Don't show important notice on password fieldTadashi G. Takaoka2014-02-171-4/+6
| | | | | | | | | | Bug: 10587358 Change-Id: Ieff7960e5e2b8609fddeb173af578a5d9b2c4d98
* | Add go to "Settings" button to important notice dialogTadashi G. Takaoka2014-02-171-10/+11
|/ | | | | | | | | | | This CL also - removes icons on important notice title. - changes the "Personalized suggestions" summary text. This change must be checked in together with Id115d89ba9. Bug: 10587358 Change-Id: I52ff26fa8ae12445e9014ba08253f69e1be609f4
* Add PunctuationSuggestions classTadashi G. Takaoka2014-02-141-3/+1
| | | | | | | This change must be checked in together with I366c477516. Bug: 12640318 Change-Id: I8d3864b9d9649c9e3306e82b01a6327b11281b1f
* Merge "Show the suggestion strip for the important notice"Jean Chalard2014-02-141-1/+4
|\
| * Show the suggestion strip for the important noticeJean Chalard2014-02-121-1/+4
| | | | | | | | | | Bug: 10587358 Change-Id: Ia060a9ca9dfc2e7fe0750536a9fea3b9cc5900c4
* | Dump settings.Jean Chalard2014-02-121-7/+1
|/ | | | | Bug: 10315296 Change-Id: Ieb020a23886514b3ba97611d8a6001f32b322f63
* Make importance notice dialog cancel ableTadashi G. Takaoka2014-02-121-1/+1
| | | | | Bug: 10587358 Change-Id: I7415a6f33e6b5542718164fea1555095d23233e5
* Show important notice stripTadashi G. Takaoka2014-02-101-2/+11
| | | | | Bug: 10587358 Change-Id: I1af3faa59afb2fc4cb90176acff1c2b0b1ba369c
* Merge "Conditionally store the raw suggestions."Jean Chalard2014-02-061-2/+2
|\
| * Conditionally store the raw suggestions.Jean Chalard2014-02-061-2/+2
| | | | | | | | | | Bug: 12913520 Change-Id: Ie15f97d90c142c51089280252b35b5ad1d226e19
* | Unregister DictionaryDumpBroadcastReceiver in onDestroy().Keisuke Kuroyanagi2014-02-061-0/+1
|/ | | | Change-Id: I83a05a8369c2fe2fa7522f18eb00983a0b10dc19
* [IL109] Move more code to InputLogicJean Chalard2014-02-051-95/+12
| | | | | Bug: 8636060 Change-Id: Ib12eacc836e8d39c4165c97a2a287f32f6bf5405
* [IL108] Remove some uses of LatinIME in input logic.Jean Chalard2014-02-051-1/+2
| | | | | | | Access the suggestion strip through the interface instead. Bug: 8636060 Change-Id: Ia8bd705c230f943e309d677ef683b1a2cbe68036