| Commit message (Expand) | Author | Age | Files | Lines |
... | |
| * | | | Fix a bug where changing auto-shift would cancel autorepeat.•••Bug: 12453966
Change-Id: Idd65f732e2de0a7309e08315893c28cca63732c5
|  Jean Chalard | 2014-01-09 | 2 | -3/+3 |
* | | | | Merge "Make reloadMainDict() private." |  Keisuke Kuroyanagi | 2014-01-08 | 2 | -26/+74 |
|\ \ \ \ |
|
| * | | | | Make reloadMainDict() private.•••Bug: 8187060
Change-Id: I852a751d748f39512e62c9cd69748dfbe87b2151
|  Keisuke Kuroyanagi | 2014-01-08 | 2 | -26/+74 |
| | |_|/
| |/| | |
|
* | | | | Revert "Revert "Separate spacing and punctuation related settings values""•••This reverts commit 8bc427799a9b7a53103d1edba4a447033fbd8cfa.
Change-Id: I34f509998afdb1a7c120fbfdb9f0a1b188224a4d
|  Jean Chalard | 2014-01-08 | 6 | -85/+153 |
| |/ /
|/| | |
|
* | | | Fix a race condition, take 2.•••Don't use absolute cursor positions when making edits,
this leads to race conditions.
This is a bit ugly and will need to be fixed soon. Plans are
underway to clean this up.
Bug: 12390573
Change-Id: I69c09fc41b979880d0800c55a710e39373287cff
|  Jean Chalard | 2014-01-08 | 2 | -5/+18 |
* | | | Merge "Revert "Separate spacing and punctuation related settings values"" |  Jean Chalard | 2014-01-08 | 6 | -153/+85 |
|\ \ \ |
|
| * | | | Revert "Separate spacing and punctuation related settings values"•••This is conflicting with later changes. Temporary revert for cherry-pick.
This reverts commit 0b593ce858d4b406e949f0e809cb77f41da691f9.
Change-Id: Id53eadb023a950cfcca496c0cfbfe583c7ec7b8c
|  Jean Chalard | 2014-01-08 | 6 | -153/+85 |
* | | | | Merge "Revert "Fix a race condition, take 2."" |  Jean Chalard | 2014-01-08 | 2 | -18/+5 |
|\ \ \ \ |
|
| * | | | | Revert "Fix a race condition, take 2."•••Conflicts prevent this to be cherry-picked.
This reverts commit dd3d697a6e6713d82020dd63cbf78e4f87664484.
Change-Id: Ib97fae2234633b4bb27d611f48a79060db9ab16f
|  Jean Chalard | 2014-01-08 | 2 | -18/+5 |
* | | | | | Merge "Fix a race condition, take 2." |  Jean Chalard | 2014-01-08 | 2 | -5/+18 |
|\| | | | |
|
| * | | | | Fix a race condition, take 2.•••Don't use absolute cursor positions when making edits,
this leads to race conditions.
This is a bit ugly and will need to be fixed soon. Plans are
underway to clean this up.
Bug: 12390573
Change-Id: Ib42d4149343c642b1b5c1937b424e8afdbd4cc1f
|  Jean Chalard | 2014-01-08 | 2 | -5/+18 |
* | | | | | Merge "Revert "Revert "Fix a race condition.""" |  Jean Chalard | 2014-01-08 | 1 | -2/+5 |
|\| | | | |
|
| * | | | | Revert "Revert "Fix a race condition.""•••This reverts commit 38401272446dbaec785e656b921cfbd31162ba09.
Change-Id: I7f0d975ec9e18d94fe3505250360c2b79ccbe96d
|  Jean Chalard | 2014-01-08 | 1 | -2/+5 |
| | |/ /
| |/| | |
|
* | | | | Rename InputPointers.addPointer and ResizableIntArray.add•••Change-Id: I5580250c91b29f93ed886b080ce33845b3b67ace
|  Tadashi G. Takaoka | 2014-01-08 | 4 | -17/+15 |
* | | | | Fix ResizableIntArrayTests readability•••Change-Id: I68862b98f228210f1d76fef4fd7454edbfd8e01d
|  Tadashi G. Takaoka | 2014-01-08 | 1 | -0/+1 |
* | | | | Merge "Fix InputPointersTests" |  Tadashi G. Takaoka | 2014-01-08 | 1 | -22/+27 |
|\ \ \ \
| |/ / /
|/| | | |
|
| * | | | Fix InputPointersTests•••InputPointers.getTime(int) has a validity check of time values. And
the check is enabled when LatinImeLogger.sDBG is on. Such situation
may occur while unit testing. This change ensure that time values are
monotonic while unit testing.
Change-Id: I9ff2cff2bcd253de0e8206dd3be964fe565170fa
|  Tadashi G. Takaoka | 2014-01-08 | 1 | -22/+27 |
* | | | | Fix: NPE in LatinIME.loadSettnigs().•••Change-Id: Icae6c74401a7a8f4da6015737f09f557640c0cca
|  Keisuke Kuroyanagi | 2014-01-08 | 1 | -2/+4 |
|/ / / |
|
* | | | Merge "Separate spacing and punctuation related settings values" |  Tadashi G. Takaoka | 2014-01-08 | 6 | -85/+153 |
|\| | |
|
| * | | Separate spacing and punctuation related settings values•••Bug: 12179576
Change-Id: Ibafd824a0ea2251843b388e95dcbd8f9bca92d5c
|  Tadashi G. Takaoka | 2014-01-08 | 6 | -85/+153 |
| |/ |
|
* | | Prepare using dictionaryFacilitator for amanatto updates.•••Bug: 11736529
Change-Id: I0d9d9b1c1108b11064163df7c2089f53e92cd7ad
|  Keisuke Kuroyanagi | 2014-01-07 | 2 | -4/+12 |
* | | Update personalized dict only when single subtype is used.•••Bug: 12424891
Change-Id: If041d1ab45988dbf9b878ae3367fee0fbc8a8988
|  Keisuke Kuroyanagi | 2014-01-07 | 3 | -3/+19 |
|/ |
|
* | Fix possible NPE.•••Bug: 11933670
Change-Id: Ib0ce853e691bdadffb73874ea9f188cfbec8bc93
|  Keisuke Kuroyanagi | 2014-01-06 | 3 | -2/+19 |
* | Merge "Add description of disabled voice input key option" |  Tadashi G. Takaoka | 2014-01-06 | 1 | -2/+8 |
|\ |
|
| * | Add description of disabled voice input key option•••Bug: 12377179
Change-Id: I44d35c6fa82883710674ecca6f630d09076472fe
|  Tadashi G. Takaoka | 2014-01-06 | 1 | -2/+8 |
* | | Merge "Update shortcut IME info before checking availability of it" |  Tadashi G. Takaoka | 2014-01-06 | 1 | -2/+6 |
|\| |
|
| * | Update shortcut IME info before checking availability of it•••Bug: 12377179
Change-Id: I4e2de78dc7e21f3f34eb56c790160b77cfc7584c
|  Tadashi G. Takaoka | 2014-01-06 | 1 | -2/+6 |
* | | Merge "Add DictionaryFacilitatorForSuggest constructor for testing" |  Tadashi G. Takaoka | 2014-01-06 | 3 | -11/+51 |
|\ \
| |/
|/| |
|
| * | Add DictionaryFacilitatorForSuggest constructor for testing•••This change must be checked in together with I00b6e79d6b.
Change-Id: Id5f87a8d0286eb44958381fefd428f16896dc0d4
|  Tadashi G. Takaoka | 2014-01-06 | 3 | -11/+51 |
* | | [IL52] Remove a useless method.•••This old method doesn't even re-read the old suggestions. It used to
recompute them without the coordinates.
Re-using the recorrection code, which is much more advanced, is
the right thing to do here.
Also, refining the test. It's no use trying to resume suggestion
if we don't have a suggestion strip, since we aren't going to
auto-correct anything anyway.
Not the motivation for this change, but this also fixes
Bug: 11620256
Change-Id: Id49efa32e293c49837c61fdc752c86bbac1d2c88
|  Jean Chalard | 2013-12-27 | 3 | -78/+25 |
* | | [IL51] Make private what can be made private•••Bug: 8636060
Change-Id: Ifa027de465a421b67897d51e9ad47666dda117ad
|  Jean Chalard | 2013-12-27 | 1 | -6/+4 |
* | | [IL50] Move the InputUpdater out of LatinIME.•••...at last
Bug: 8636060
Change-Id: If5e27a9d28ff64d318dd76792f55edfc8a78d2d7
|  Jean Chalard | 2013-12-27 | 3 | -201/+168 |
* | | [IL49] Move InputLogic-related init code to InputLogic.•••Bug: 8636060
Change-Id: Ie5027512f066d9c530a686ac559e4649a39bf439
|  Jean Chalard | 2013-12-27 | 2 | -19/+22 |
* | | Fix a failing test•••How/when did this happen? How was it not noticed?
Change-Id: I352a8dad65f8c27bf5120d567e96d9a12cfa8496
|  Jean Chalard | 2013-12-27 | 1 | -2/+3 |
* | | Merge "[IL48] Move onEndBatchInputAsyncInternal to InputLogic." |  Jean Chalard | 2013-12-27 | 2 | -35/+43 |
|\ \ |
|
| * | | [IL48] Move onEndBatchInputAsyncInternal to InputLogic.•••Bug: 8636060
Change-Id: Idfabf7231a80e21786ce1910b5140bb3dddcac7d
|  Jean Chalard | 2013-12-27 | 2 | -35/+43 |
* | | | Merge "[IL47] Inline getSuggestedWordsGestureLocked" |  Jean Chalard | 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
|  Jean Chalard | 2013-12-27 | 1 | -23/+10 |
* | | | Merge "[IL46] Remove a useless message passing." |  Jean Chalard | 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
|  Jean Chalard | 2013-12-27 | 1 | -20/+2 |
* | | | Merge "[IL45] Remove lingering messages" |  Jean Chalard | 2013-12-27 | 1 | -4/+9 |
|\| | |
|
| * | | [IL45] Remove lingering messages•••Bug: 8636060
And possibly bug: 12085864
Change-Id: I867036bc3d8bd255dc8ceebdd30e28b4b3dc4b2d
|  Jean Chalard | 2013-12-27 | 1 | -4/+9 |
* | | | Merge "[IL44] Some factorization" |  Jean Chalard | 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
|  Jean Chalard | 2013-12-27 | 2 | -27/+63 |
* | | Merge "Make setAdditionalDictionaries private." |  Keisuke Kuroyanagi | 2013-12-27 | 3 | -52/+91 |
|\ \ |
|
| * | | Make setAdditionalDictionaries private.•••Bug: 8187060
Change-Id: I17be7b77ff89972d3ca6823b087b7bf35105af56
|  Keisuke Kuroyanagi | 2013-12-27 | 3 | -52/+91 |
* | | | Fix possible NPE in FileUtils.•••Change-Id: I503f91e266c71e2370a5807d171e2254c334f7cb
|  Keisuke Kuroyanagi | 2013-12-27 | 1 | -2/+8 |
| |/
|/| |
|
* | | Fix a bug where predictions would not pop upon manual pick•••Bug: 12295276
Change-Id: Id359c92acde44758b12929e7bac719d5c9c7577b
|  Jean Chalard | 2013-12-26 | 2 | -3/+6 |
* | | Merge "Fix a code path that looks ridiculous" |  Jean Chalard | 2013-12-26 | 2 | -12/+12 |
|\ \ |
|
| * | | Fix a code path that looks ridiculous•••Bug: 12295276
Change-Id: I187016b59213d351c943c6e0b9a2ad3708499646
|  Jean Chalard | 2013-12-26 | 2 | -12/+12 |
| |/ |
|