aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin/LatinIME.java (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fix a bug on rotation with selection."Jean Chalard2014-06-061-5/+1
|\
| * Fix a bug on rotation with selection.Jean Chalard2014-06-061-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The symptom : when text is selected and the device is rotated, sometimes the keyboard sets the word as being composed around the start of the selection. Upon the next rotation this ends up with the keyboard committing some text in place of the selection. The cause : another bug in the framework with rotation >.> The keyboard receives a call to startInput with a wrong cursor position, namely one that does not represent a selection. The keyboard sets a composition according to this wrong data. When the keyboard is rotated again, it commits the text, which takes the place of the selection. The solution : actually when restarting input the keyboard realizes that the cursor position is wrong. We cancel composition at that time. For robustness, this change also implements two other defensive changes : upon call to onUpdateSelection, we actually realize that the previous values were wrong, so we also fix it at that time, and in addition, when rotating, we finishComposingText() instead of commitText() which is less dangerous. Implementing this later change also allows us to let less internal variables from InputLogic escape to LatinIME, so it's also a good change for design. Bug: 14140799 Change-Id: Ib10de18e53e376ac1bbc8487e13d969828483346
* | Rename some boolean variables for readabilityTadashi G. Takaoka2014-06-051-3/+3
|/ | | | Change-Id: I434cf507e68349b9893ac084645986d6b3d7da91
* Remove LatinImeLogger and UsabilityStudyLogUtilsTadashi G. Takaoka2014-05-291-8/+0
| | | | | | | This CL must be checked in together with Ie52007dc5c. Bug: 15318007 Change-Id: Ie665f296e43b5485ae8ba7e949aa262794ee0dcc
* Remove researcher loggerTadashi G. Takaoka2014-05-291-30/+0
| | | | | | | This CL must be checked in together with I5cc76807e3. Bug: 15318007 Change-Id: I61423c3377ddc299fb332e742d6626c2e47145bb
* Keep showing important notice when "show correction suggestions" is offTadashi G. Takaoka2014-05-261-4/+11
| | | | | | | | | | | When 1. The important notice message is shown on the suggestions strip. 2. And the "Show correction suggestions" settings is off. we will keep showing the important notice message on the suggestion strip. Bug: 13741460 Change-Id: I411007ab1e5e6959b6cdba7a6601a84635259313
* Use Java 7 diamond operatorTadashi G. Takaoka2014-05-241-3/+1
| | | | Change-Id: If16ef50ae73147594615d0f49d6a22621eaf1aef
* Make DistracterFilter interface.Keisuke Kuroyanagi2014-05-231-2/+2
| | | | | | Bug: 13142176 Bug: 15094186 Change-Id: If94b0a155b5ea2ff6b839e7da9d12a9cc6553931
* Move distracter filter to dictionary facilitator.Keisuke Kuroyanagi2014-05-231-11/+11
| | | | | | Bug: 13142176 Bug: 15094186 Change-Id: Ib9e42f8e25538009f3ba62cf1e034cae3f6823c7
* Have DictionaryFacilitator in LatinIME.Keisuke Kuroyanagi2014-05-231-30/+21
| | | | | | Bug: 13142176 Bug: 15094186 Change-Id: I9da67352ff2abcd289fa5885389895ced878a413
* Fix NPE happening in unit tests.Keisuke Kuroyanagi2014-05-221-2/+3
| | | | Change-Id: I2fc0dba53e18883755ff77351c152a9d3534c92b
* Merge "Use enabled subtypes for Keyboard in DistracterFilter."Keisuke Kuroyanagi2014-05-221-2/+3
|\
| * Use enabled subtypes for Keyboard in DistracterFilter.Keisuke Kuroyanagi2014-05-211-2/+3
| | | | | | | | | | | | Bug: 13142176 Bug: 15094186 Change-Id: I142eeaa11f838080a4fc5a342155e5d470eab3d4
* | Add beginning of sentence information in PrevWordsInfo.Keisuke Kuroyanagi2014-05-211-5/+6
|/ | | | | | Bug: 14119293 Bug: 14425059 Change-Id: I65320920e840082b0b697bb621676716d0933e0c
* Use PrevWordsInfo instead of String in Java side.Keisuke Kuroyanagi2014-05-191-5/+5
| | | | | | | Bug: 14119293 Bug: 14425059 Change-Id: I3d5da84881a49a04550180dd9aac2c37da2ed762
* Use dedicated DictionaryFacilitator for DistracterFilterKeisuke Kuroyanagi2014-05-171-5/+5
| | | | | Bug: 15002249 Change-Id: Ic77eaa7d627d406daad7dcd15cb4884d9f647312
* Revert "Use dedicated DictionaryFacilitator for DistracterFilter"Keisuke Kuroyanagi2014-05-161-5/+5
| | | | | | This reverts commit 7122fe76e3ca48a12f1e069ee0c742788c1884b6. Change-Id: I341089864a2b58de6e0f35844a12fb632bd1eabb
* Use dedicated DictionaryFacilitator for DistracterFilterKeisuke Kuroyanagi2014-05-161-5/+5
| | | | | Bug: 15002249 Change-Id: I2ebf1941fb0453ea4b97c0e539f8b36e5e0262c1
* Remove unused auto correction led for spacebar featureTadashi G. Takaoka2014-05-151-2/+0
| | | | Change-Id: Ie1544bd0d28684d9b7887fddff37b224ed4f8201
* Merge "Simplify StatsUtils"Yohei Yukawa2014-05-151-1/+3
|\
| * Simplify StatsUtilsYohei Yukawa2014-05-151-1/+3
| | | | | | | | | | | | | | | | | | | | | | With this CL, the implementation of StatsUtils no longer needs to know how to read settings from the system. Insted, the LatinIME class is now responsible for notifying StatsUtils whenever the settings is changed. BUG: 14324207 Change-Id: Ic3d26ec31c8d2c082d3e7487b578b323aad2f960
* | Fix off-device buildXiaojun Bi2014-05-141-1/+3
|/ | | | | | | | | This CL fixed the breakage caused by DistracterFilter. It should be checked in together with I8f53e9481c0f Bug: 14911612 Bug: 13142176 Change-Id: I33c3526165cea58926d10404552f1fadc385c2e5
* Merge "Implement the distracter filter"Xiaojun Bi2014-05-131-12/+5
|\
| * Implement the distracter filterXiaojun Bi2014-05-131-12/+5
| | | | | | | | | | | | Bug: 13142176 Change-Id: I85a048cdcbbad3d6020a25f900250d8d7a0fe3da
* | Create the path for specifying combining rules in the subtypeJean Chalard2014-05-121-1/+5
|/ | | | | | | | Also add the skeleton of the combining rules for Myanmar reordering as a pilot. Bug: 13945569 Change-Id: Iabeb6838f45f1efff482143595ce5b48543e0c43
* Merge "Make AccessibleKeyboardViewProxy not singleton"Tadashi G. Takaoka2014-05-011-5/+0
|\
| * Make AccessibleKeyboardViewProxy not singletonTadashi G. Takaoka2014-05-011-5/+0
| | | | | | | | | | | | | | | | This CL also renames AccessibleKeyboardViewProxy and AccessibilityEntityProvider to MainKeyboardAccessibilityDelegate and MainKeyboardAccessibilityNodeProvider. Change-Id: I2b0ec091a11aa8a495794d633efecb6d8b818f42
* | Merge "Handle user dict as an ExpandableBinaryDictionary."Keisuke Kuroyanagi2014-04-301-1/+2
|\ \ | |/ |/|
| * Handle user dict as an ExpandableBinaryDictionary.Keisuke Kuroyanagi2014-04-301-1/+2
| | | | | | | | | | Bug: 13755213 Change-Id: I2d242a3332e9633d4e41e08c0bbfafcefd814ecd
* | TalkBack correct shift stateTadashi G. Takaoka2014-04-301-12/+0
| | | | | | | | Change-Id: I3097805c6c8b4a4ac19fbe062f763e2639eb5cbf
* | Use shouldOfferSwitchingToNextInputMethod when availableYohei Yukawa2014-04-281-1/+25
|/ | | | | | | | | | | | | | | | With this CL, LatinIME starts using InputMethodManager#shouldOfferSwitchingToNextInputMethod when available and API level is higher than 19 (KitKat). Note that relevant settings of LatinIME will be ignored if InputMethodManager#shouldOfferSwitchingToNextInputMethod is considered to be available at the moment. We will revisit here to reorganize the user visible settings before the new global IME switching mechanism becomes publicly available. BUG: 12965588 Change-Id: I0188fa56cba8e983c61cef3ae3400a0e3821f718
* Add null check in createDistracterFilter().Keisuke Kuroyanagi2014-04-251-1/+5
| | | | Change-Id: Iff2b73081d48f32b1d3a5d5b32c9734979362c23
* Prepare to implement distracter filterXiaojun Bi2014-04-251-2/+14
| | | | | | | | | | | | | | | It was implemented according to the Plan B in the design doc: http://go/ime-misspelling-filter This CL is to create a DistracterFilter instance and pass it to PersonalizationDictionarySessionRegistrar. This patch should be checked in together with Id633b4fd45693 Bug: 13142176 Change-Id: Ia4957e64218c9619fcf9bb91795a187617c72a2e
* Merge "Fix a wrong parameter passed."Jean Chalard2014-04-171-1/+1
|\
| * Fix a wrong parameter passed.Jean Chalard2014-04-171-1/+1
| | | | | | | | | | Bug: 14120001 Change-Id: I8258304bc500fe28870fb8a5c11c15d97795423c
* | Merge "Restart suggestions when restarting input"Jean Chalard2014-04-161-3/+1
|\ \
| * | Restart suggestions when restarting inputJean Chalard2014-04-161-3/+1
| |/ | | | | | | | | Bug: 6851364 Change-Id: I3052924053f37f56c9c3953051aa1b22224a2b2e
* / Fix some failing testsJean Chalard2014-04-151-3/+6
|/ | | | | Bug: 13968927 Change-Id: I7e5e075535e38bf0fa24af98ed6e2cdfd6de6556
* Merge "[IL135] Make getCurrentAutoCapsState private"Jean Chalard2014-04-151-16/+18
|\
| * [IL135] Make getCurrentAutoCapsState privateJean Chalard2014-04-141-16/+18
| | | | | | | | | | Bug: 8636060 Change-Id: I800aefe6fee4c14df6fe5abe21b2387c69abb8f5
* | Merge "[IL134] Add an alternative path to getCurrentAutoCapsState"Jean Chalard2014-04-151-1/+2
|\|
| * [IL134] Add an alternative path to getCurrentAutoCapsStateJean Chalard2014-04-141-1/+2
| | | | | | | | | | | | Bug: 8636060 Change-Id: I959e13d34910adad028ed6174f92029ff690eeeb
* | Fix updating the shift state upon backspaceJean Chalard2014-04-141-4/+5
|/ | | | | Bug: 13514349 Change-Id: If4c9db12b0ab5be676f7a2f72715f469066ee537
* Keep ImportantNoticeDialog showing after orientation changedTadashi G. Takaoka2014-04-141-9/+6
| | | | | Bug: 13951402 Change-Id: Ia6340d48843abca68ebadd299827cbe43516cc39
* Merge "Plumbing for ProductionFlag.USES_CURSOR_ANCHOR_MONITOR"Yohei Yukawa2014-04-131-0/+14
|\
| * Plumbing for ProductionFlag.USES_CURSOR_ANCHOR_MONITORYohei Yukawa2014-04-111-0/+14
| | | | | | | | | | | | | | | | | | | | With this CL, LatinIME starts calling InputMethodService#setCursorAnchorMonitorMode in #onStartInput() when ProductionFlag.USES_CURSOR_ANCHOR_MONITOR flag is explicitly set to true. BUG: 13891796 Change-Id: Ib2fb0c3521b61859d4cc530155ccaaee7ee16cbc
* | Merge "[IL133] Avoid calling a dangerous method from outside."Jean Chalard2014-04-101-5/+10
|\ \
| * | [IL133] Avoid calling a dangerous method from outside.Jean Chalard2014-04-031-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | We want to remove all calls to this as it lets internal values escape, but there is some refactoring to do to finish this. Bug: 8636060 Change-Id: Iedba6afe4719bc0add868714a1ee5a04b7ead33e
* | | Fix some flaky tests.Jean Chalard2014-04-101-0/+5
| |/ |/| | | | | | | | | | | | | | | | | In tests, we create many instances of LatinIME, but we never destroy them. That means we never close the dictionaries nor the handlers. This change calls onDestroy, which closes all dictionaries, and adds some code to finish the handlers. Change-Id: I942517a2a940c54256b08763f6b38f5b55809f55
* | Merge "Add StatsUtils for checking state"Yohei Yukawa2014-04-081-0/+4
|\ \