aboutsummaryrefslogtreecommitdiffstats
path: root/java/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
| * | [IL47] Inline getSuggestedWordsGestureLockedJean Chalard2013-12-271-23/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | This method is confusing with the *Locked convention, and the two-step call creates a useless callback object. This is better inlined both for readability and for performance. Bug: 8636060 Change-Id: I7c427c3ca4e831388a6d54de6728b32206a45d80
* | | Merge "[IL46] Remove a useless message passing."Jean Chalard2013-12-271-20/+2
|\| |
| * | [IL46] Remove a useless message passing.Jean Chalard2013-12-271-20/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only point of this message is to send the processing on another thread. However, this will be accomplished later. Here is the exact call graph: 0. onUpdateBatchInput 1. -> MSG_UPDATE_GESTURE_PREVIEW_AND_SUGGESTION_STRIP 2. -> updateBatchInputSync 3. -> getSuggestedWordsGestureLocked 4. -> MSG_GET_SUGGESTED_WORDS 5. -> LatinIME#getSuggestedWords The point of both step 1. and step 4. is to make sure the processing is happening on the InputUpdater thread. Thus, it's useless to do it twice. Bug: 11326092 Bug: 8636060 Change-Id: Iceebb9e8879a8f15b73c987f5fd3489f27699be4
* | | Merge "[IL45] Remove lingering messages"Jean Chalard2013-12-271-4/+9
|\| |
| * | [IL45] Remove lingering messagesJean Chalard2013-12-271-4/+9
| | | | | | | | | | | | | | | | | | | | | Bug: 8636060 And possibly bug: 12085864 Change-Id: I867036bc3d8bd255dc8ceebdd30e28b4b3dc4b2d
* | | Merge "[IL44] Some factorization"Jean Chalard2013-12-272-27/+63
|\| | | |/ |/|
| * [IL44] Some factorizationJean Chalard2013-12-272-27/+63
| | | | | | | | | | | | | | | | | | This sheds some light on what's happening here. Some comments were at least misleading, maybe indicating something is not sequenced as intended. Bug: 8636060 Change-Id: Ia74feb457a39fe4a672c27fe4203264fda940f04
* | Merge "Make setAdditionalDictionaries private."Keisuke Kuroyanagi2013-12-273-52/+91
|\ \
| * | Make setAdditionalDictionaries private.Keisuke Kuroyanagi2013-12-273-52/+91
| | | | | | | | | | | | | | | Bug: 8187060 Change-Id: I17be7b77ff89972d3ca6823b087b7bf35105af56
* | | Fix possible NPE in FileUtils.Keisuke Kuroyanagi2013-12-271-2/+8
| |/ |/| | | | | Change-Id: I503f91e266c71e2370a5807d171e2254c334f7cb
* | Fix a bug where predictions would not pop upon manual pickJean Chalard2013-12-262-3/+6
| | | | | | | | | | Bug: 12295276 Change-Id: Id359c92acde44758b12929e7bac719d5c9c7577b
* | Merge "Fix a code path that looks ridiculous"Jean Chalard2013-12-262-12/+12
|\ \
| * | Fix a code path that looks ridiculousJean Chalard2013-12-262-12/+12
| |/ | | | | | | | | Bug: 12295276 Change-Id: I187016b59213d351c943c6e0b9a2ad3708499646
* | [IL43] Small cleanupJean Chalard2013-12-262-9/+8
| | | | | | | | | | | | | | All calls are already passing this Bug: 8636060 Change-Id: I13accefd7c17039dc5ec12dc9e8133ebc86451ad
* | [IL42] Inline getSuggestedWordsOrOlderSuggestionsAsyncJean Chalard2013-12-262-39/+34
| | | | | | | | | | Bug: 8636060 Change-Id: I5c47370a3ea5962caff4d40db2faa62accfc7147
* | [IL41] Remove a useless call and trampoline.Jean Chalard2013-12-261-14/+14
|/ | | | | | | | No point in calling maybeRetrieveOlderSuggestions now that the same work is done right here. The callback is also useless now. Bug: 8636060 Change-Id: Ic14849cb6f6e6293a42cfe068f5c6b18b7677bc0
* Remove DictionaryFacilitatorForSuggest.getMainDictionary().Keisuke Kuroyanagi2013-12-263-30/+20
| | | | | Bug: 8187060 Change-Id: Ide087f1e39b24695b610bd2b3c464b7e3cb9cd29
* Extract dict operations from Suggest to a new class.Keisuke Kuroyanagi2013-12-266-360/+426
| | | | | Bug: 8187060 Change-Id: I77775aa50763158d99753c2312fa11fe14267aef
* [IL39] Change a test.Jean Chalard2013-12-261-5/+5
| | | | | | | | | I don't see why we would want to remove other suggestions just because the only suggestion happens to be a dictionary word. That's just inconsistent. Bug: 8636060 Change-Id: Ie2d990e5785273d625e6e5518026364165a8f2df
* [IL38] Fix a wrong double callJean Chalard2013-12-261-7/+7
| | | | | | This only gives bad results if a gesture results in only one suggestion. Change-Id: I5a2f27f83f6901a5a9ecf22f02bdfc9cca6d9f1d
* Merge "Remove useless method call for spell checker."Keisuke Kuroyanagi2013-12-263-25/+22
|\
| * Remove useless method call for spell checker.Keisuke Kuroyanagi2013-12-263-25/+22
| | | | | | | | | | Bug: 8187060 Change-Id: I2f91da12ba2a9e7ce28abd06f314e0e6125d08dc
* | Merge "Add BatchInputArbiter"Tadashi G. Takaoka2013-12-264-109/+271
|\ \
| * | Add BatchInputArbiterTadashi G. Takaoka2013-12-264-109/+271
| | | | | | | | | | | | | | | | | | | | | | | | | | | Move the logic that determines how to handle gesture typing from PointerTracker to BatchInputArbiter class. This change must be checked in together with Ia7db8f4fb7. Change-Id: I5e9f786de58d8a3cb0407ae562af707710ec37fb
* | | [IL37] Small simplificationJean Chalard2013-12-263-12/+7
| | | | | | | | | | | | | | | Bug: 8636060 Change-Id: I08a6c04986a0bd86ce4c232893e804766ebeebb9
* | | [IL36] Inline setAutoCorrectionJean Chalard2013-12-261-14/+9
| | | | | | | | | | | | | | | | | | | | | There are far too many one-use methods. Bug: 8636060 Change-Id: I84e0669b8d2cc5a2817c42b089e874dc60e84769
* | | [IL35] Preparative cleanupJean Chalard2013-12-252-13/+14
| | | | | | | | | | | | | | | Bug: 8636060 Change-Id: Id6791085816891c78289b0223bf0807db2da6300
* | | Merge "[IL34] Introduce InputLogicHandler"Jean Chalard2013-12-252-0/+55
|\ \ \ | |_|/ |/| |
| * | [IL34] Introduce InputLogicHandlerJean Chalard2013-12-252-0/+55
| | | | | | | | | | | | | | | Bug: 8636060 Change-Id: I5d08ed5d69cdb03bd188e36b7014de03f04fa72f
* | | Remove "Boost personalization dictionary" from settings.Keisuke Kuroyanagi2013-12-251-2/+0
| | | | | | | | | | | | Change-Id: Iab05e42e947610f358472cd95d0319d0ab3f0b31
* | | Use SettingsValues instead of Settings in Suggest.Keisuke Kuroyanagi2013-12-254-29/+7
| | | | | | | | | | | | | | | Bug: 8187060 Change-Id: Idba1e708d67c036072b728949d17f8bd98ae0115
* | | Merge "Separate GestureStrokeDrawingPoints from GestureStrokeRecognitionPoints"Tadashi G. Takaoka2013-12-253-39/+37
|\ \ \ | | |/ | |/|
| * | Separate GestureStrokeDrawingPoints from GestureStrokeRecognitionPointsTadashi G. Takaoka2013-12-253-39/+37
| | | | | | | | | | | | | | | | | | This change must be checked in together with Ibed5330614. Change-Id: I20ba79fd20e8ea0aa3ac596c011ec5bfea528ace
* | | Move dict operations to Suggest.Keisuke Kuroyanagi2013-12-254-98/+96
| |/ |/| | | | | | | | | Bug: 8187060 Change-Id: Iabe13abcd43b381360f7080b3f4ee0533f246160
* | Fix NPE in SettingsValues.Keisuke Kuroyanagi2013-12-251-1/+1
| | | | | | | | Change-Id: I96fc381d2b45881aeb342d33207cd7fe82f5077c
* | Merge "Rename gesture related classes"Tadashi G. Takaoka2013-12-258-155/+228
|\|
| * Rename gesture related classesTadashi G. Takaoka2013-12-258-155/+228
| | | | | | | | Change-Id: I5cb03576bb7221f1864e157857d872880a0a58f8
* | Remove personalized dicts when the option is turned off.Keisuke Kuroyanagi2013-12-257-11/+85
|/ | | | | | Bug: 11757851 Change-Id: I94f32859d862fe41e39845ab2bfc2d396c6aea0f
* Comprehensive memory deallocationTadashi G. Takaoka2013-12-246-13/+21
| | | | Change-Id: I4673543c96eb342c3c977e00d44772a0d719a23e
* Rename classes related to AbstractDrawingPreviewTadashi G. Takaoka2013-12-246-50/+53
| | | | Change-Id: Ife2e2c08b2b6fbf9ec33cf259f276878816b291c
* Separate GestureStrokePreviewParams to top level classTadashi G. Takaoka2013-12-243-37/+51
| | | | Change-Id: I4c6ac12dac9636760128625bb5b7138b7d56f3d1
* Merge "Make GestureStrokeParams as a top level class"Tadashi G. Takaoka2013-12-243-75/+94
|\
| * Make GestureStrokeParams as a top level classTadashi G. Takaoka2013-12-243-75/+94
| | | | | | | | Change-Id: I540a2d3a6c6ce6ecdcccecd38df6b3c23fcf8d95
* | Merge "[IL33] Opportunistically remove some KeyboardSwitcher"Jean Chalard2013-12-242-22/+20
|\ \ | |/ |/|
| * [IL33] Opportunistically remove some KeyboardSwitcherJean Chalard2013-12-242-22/+20
| | | | | | | | | | Bug: 8636060 Change-Id: I0480ac03b9902e940f566b5558b8499647983968
* | Clean up batch input related code a bitTadashi G. Takaoka2013-12-241-54/+60
| | | | | | | | Change-Id: I3b6a95fdb82bd15e2b636b1374a96d8987d5d8cf
* | Remove duplicate codeTadashi G. Takaoka2013-12-241-1/+1
|/ | | | Change-Id: Ia33ea2dc4bb063ec5bf282c656ccc1f11ac967fe
* [IL32] Remove a useless variableJean Chalard2013-12-201-3/+2
| | | | Change-Id: I8f5d8fd8c1627a568df57255dc3dceda26c4720c
* [IL31] Move *BatchInput to InputLogicJean Chalard2013-12-202-107/+128
| | | | | Bug: 8636060 Change-Id: If0309be038518a71d735108b78ac9efbfe38c82b
* [IL30] Minor refactoringJean Chalard2013-12-201-3/+3
| | | | Change-Id: I8a8aa9f703c6fca5e25ec573b41f325680ac92dc