Commit message (Expand) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | [IL50] Move the InputUpdater out of LatinIME.•••...at last Bug: 8636060 Change-Id: If5e27a9d28ff64d318dd76792f55edfc8a78d2d7 | 2013-12-27 | 3 | -201/+168 | ||
* | | [IL49] Move InputLogic-related init code to InputLogic.•••Bug: 8636060 Change-Id: Ie5027512f066d9c530a686ac559e4649a39bf439 | 2013-12-27 | 2 | -19/+22 | ||
* | | Fix a failing test•••How/when did this happen? How was it not noticed? Change-Id: I352a8dad65f8c27bf5120d567e96d9a12cfa8496 | 2013-12-27 | 1 | -2/+3 | ||
* | | Merge "[IL48] Move onEndBatchInputAsyncInternal to InputLogic." | 2013-12-27 | 2 | -35/+43 | ||
|\ \ | ||||||
| * | | [IL48] Move onEndBatchInputAsyncInternal to InputLogic.•••Bug: 8636060 Change-Id: Idfabf7231a80e21786ce1910b5140bb3dddcac7d | 2013-12-27 | 2 | -35/+43 | ||
* | | | Merge "[IL47] Inline getSuggestedWordsGestureLocked" | 2013-12-27 | 1 | -23/+10 | ||
|\| | | ||||||
| * | | [IL47] Inline getSuggestedWordsGestureLocked•••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 | 2013-12-27 | 1 | -23/+10 | ||
* | | | Merge "[IL46] Remove a useless message passing." | 2013-12-27 | 1 | -20/+2 | ||
|\| | | ||||||
| * | | [IL46] Remove a useless message passing.•••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 | 2013-12-27 | 1 | -20/+2 | ||
* | | | Merge "[IL45] Remove lingering messages" | 2013-12-27 | 1 | -4/+9 | ||
|\| | | ||||||
| * | | [IL45] Remove lingering messages•••Bug: 8636060 And possibly bug: 12085864 Change-Id: I867036bc3d8bd255dc8ceebdd30e28b4b3dc4b2d | 2013-12-27 | 1 | -4/+9 | ||
* | | | Merge "[IL44] Some factorization" | 2013-12-27 | 2 | -27/+63 | ||
|\| | | |/ |/| | ||||||
| * | [IL44] Some factorization•••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 | 2013-12-27 | 2 | -27/+63 | ||
* | | Merge "Make setAdditionalDictionaries private." | 2013-12-27 | 3 | -52/+91 | ||
|\ \ | ||||||
| * | | Make setAdditionalDictionaries private.•••Bug: 8187060 Change-Id: I17be7b77ff89972d3ca6823b087b7bf35105af56 | 2013-12-27 | 3 | -52/+91 | ||
* | | | Fix possible NPE in FileUtils.•••Change-Id: I503f91e266c71e2370a5807d171e2254c334f7cb | 2013-12-27 | 1 | -2/+8 | ||
| |/ |/| | ||||||
* | | Fix a bug where predictions would not pop upon manual pick•••Bug: 12295276 Change-Id: Id359c92acde44758b12929e7bac719d5c9c7577b | 2013-12-26 | 2 | -3/+6 | ||
* | | Merge "Fix a code path that looks ridiculous" | 2013-12-26 | 2 | -12/+12 | ||
|\ \ | ||||||
| * | | Fix a code path that looks ridiculous•••Bug: 12295276 Change-Id: I187016b59213d351c943c6e0b9a2ad3708499646 | 2013-12-26 | 2 | -12/+12 | ||
| |/ | ||||||
* | | [IL43] Small cleanup•••All calls are already passing this Bug: 8636060 Change-Id: I13accefd7c17039dc5ec12dc9e8133ebc86451ad | 2013-12-26 | 2 | -9/+8 | ||
* | | [IL42] Inline getSuggestedWordsOrOlderSuggestionsAsync•••Bug: 8636060 Change-Id: I5c47370a3ea5962caff4d40db2faa62accfc7147 | 2013-12-26 | 2 | -39/+34 | ||
* | | [IL41] Remove a useless call and trampoline.•••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 | 2013-12-26 | 1 | -14/+14 | ||
|/ | ||||||
* | Remove DictionaryFacilitatorForSuggest.getMainDictionary().•••Bug: 8187060 Change-Id: Ide087f1e39b24695b610bd2b3c464b7e3cb9cd29 | 2013-12-26 | 3 | -30/+20 | ||
* | Extract dict operations from Suggest to a new class.•••Bug: 8187060 Change-Id: I77775aa50763158d99753c2312fa11fe14267aef | 2013-12-26 | 6 | -360/+426 | ||
* | [IL39] Change a test.•••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 | 2013-12-26 | 1 | -5/+5 | ||
* | [IL38] Fix a wrong double call•••This only gives bad results if a gesture results in only one suggestion. Change-Id: I5a2f27f83f6901a5a9ecf22f02bdfc9cca6d9f1d | 2013-12-26 | 1 | -7/+7 | ||
* | Merge "Remove useless method call for spell checker." | 2013-12-26 | 3 | -25/+22 | ||
|\ | ||||||
| * | Remove useless method call for spell checker.•••Bug: 8187060 Change-Id: I2f91da12ba2a9e7ce28abd06f314e0e6125d08dc | 2013-12-26 | 3 | -25/+22 | ||
* | | Merge "Add BatchInputArbiter" | 2013-12-26 | 4 | -109/+271 | ||
|\ \ | ||||||
| * | | Add BatchInputArbiter•••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 | 2013-12-26 | 4 | -109/+271 | ||
* | | | [IL37] Small simplification•••Bug: 8636060 Change-Id: I08a6c04986a0bd86ce4c232893e804766ebeebb9 | 2013-12-26 | 3 | -12/+7 | ||
* | | | [IL36] Inline setAutoCorrection•••There are far too many one-use methods. Bug: 8636060 Change-Id: I84e0669b8d2cc5a2817c42b089e874dc60e84769 | 2013-12-26 | 1 | -14/+9 | ||
* | | | [IL35] Preparative cleanup•••Bug: 8636060 Change-Id: Id6791085816891c78289b0223bf0807db2da6300 | 2013-12-25 | 2 | -13/+14 | ||
* | | | Merge "[IL34] Introduce InputLogicHandler" | 2013-12-25 | 2 | -0/+55 | ||
|\ \ \ | |_|/ |/| | | ||||||
| * | | [IL34] Introduce InputLogicHandler•••Bug: 8636060 Change-Id: I5d08ed5d69cdb03bd188e36b7014de03f04fa72f | 2013-12-25 | 2 | -0/+55 | ||
* | | | Remove "Boost personalization dictionary" from settings.•••Change-Id: Iab05e42e947610f358472cd95d0319d0ab3f0b31 | 2013-12-25 | 3 | -10/+0 | ||
* | | | Use SettingsValues instead of Settings in Suggest.•••Bug: 8187060 Change-Id: Idba1e708d67c036072b728949d17f8bd98ae0115 | 2013-12-25 | 4 | -29/+7 | ||
* | | | Merge "Separate GestureStrokeDrawingPoints from GestureStrokeRecognitionPoints" | 2013-12-25 | 3 | -39/+37 | ||
|\ \ \ | | |/ | |/| | ||||||
| * | | Separate GestureStrokeDrawingPoints from GestureStrokeRecognitionPoints•••This change must be checked in together with Ibed5330614. Change-Id: I20ba79fd20e8ea0aa3ac596c011ec5bfea528ace | 2013-12-25 | 3 | -39/+37 | ||
* | | | Move dict operations to Suggest.•••Bug: 8187060 Change-Id: Iabe13abcd43b381360f7080b3f4ee0533f246160 | 2013-12-25 | 4 | -98/+96 | ||
| |/ |/| | ||||||
* | | Fix NPE in SettingsValues.•••Change-Id: I96fc381d2b45881aeb342d33207cd7fe82f5077c | 2013-12-25 | 1 | -1/+1 | ||
* | | Merge "Rename gesture related classes" | 2013-12-25 | 8 | -155/+228 | ||
|\| | ||||||
| * | Rename gesture related classes•••Change-Id: I5cb03576bb7221f1864e157857d872880a0a58f8 | 2013-12-25 | 8 | -155/+228 | ||
* | | Remove personalized dicts when the option is turned off.•••Bug: 11757851 Change-Id: I94f32859d862fe41e39845ab2bfc2d396c6aea0f | 2013-12-25 | 7 | -11/+85 | ||
|/ | ||||||
* | Comprehensive memory deallocation•••Change-Id: I4673543c96eb342c3c977e00d44772a0d719a23e | 2013-12-24 | 6 | -13/+21 | ||
* | Rename classes related to AbstractDrawingPreview•••Change-Id: Ife2e2c08b2b6fbf9ec33cf259f276878816b291c | 2013-12-24 | 6 | -50/+53 | ||
* | Separate GestureStrokePreviewParams to top level class•••Change-Id: I4c6ac12dac9636760128625bb5b7138b7d56f3d1 | 2013-12-24 | 3 | -37/+51 | ||
* | Merge "Make GestureStrokeParams as a top level class" | 2013-12-24 | 3 | -75/+94 | ||
|\ | ||||||
| * | Make GestureStrokeParams as a top level class•••Change-Id: I540a2d3a6c6ce6ecdcccecd38df6b3c23fcf8d95 | 2013-12-24 | 3 | -75/+94 | ||
* | | Merge "[IL33] Opportunistically remove some KeyboardSwitcher" | 2013-12-24 | 2 | -22/+20 | ||
|\ \ | |/ |/| |