| Commit message (Expand) | Author | Files | Lines |
2014-01-10 | Periodic wipe of the personalization dict.•••Bug: 12216453
Change-Id: Ie59dd19c044e9b1eb9b8bd805366200737ea506a
|  Keisuke Kuroyanagi | 3 | -1/+17 |
2014-01-10 | [IL54] Add a mirror method to fix the cursor pos estimate•••This just mirrors what InputLogic#tryFixLyingCursorPosition
is doing. That method will go away in the next change.
Change-Id: Ifa2827dbc1f1d20e2c642d6f2d23514a01ed9203
|  Jean Chalard | 2 | -5/+57 |
2014-01-10 | [IL53] Fix a wrong test.•••This test was intended only for cases without a selection, and as
a safety net for cases where the app would pretend the cursor
is at N but we can get P chars from the editor where P > N.
When there is a selection, this is wrong. In the practice it works
because these values are not used in this case, but it's still wrong.
The case where P > N is arguable, but actually I see little reason
to trust the getTextBeforeCursor() method more than the
onUpdate selection method. Plus in the practice, I don't think
we are aware of any app with this bug, and it's probably not a
great idea to be too robust about this as it may encourage wrong
values sent to onUpdateSelection.
Change-Id: I42f2065d7aee668074e6b8e40b259da7e88e16e1
|  Jean Chalard | 1 | -5/+7 |
2014-01-10 | Fix an NPE.•••Bug: 12397228
Change-Id: I7632931f0685fc8f0558946be66025b2bb2b5a3d
|  Jean Chalard | 2 | -3/+31 |
2014-01-10 | Fix build.•••Change-Id: I6d0b572190080e58e95f72856ad39e72a49537b7
|  Jean Chalard | 5 | -70/+75 |
2014-01-09 | Passing SpacingAndPunctuations to StringUtils.looksValidForDictionaryInsertion•••This change must be checked in together with Iec437d6df0.
Change-Id: Idf745b8870965f3bdf690de7ff3b22f83f6204bf
|  Tadashi G. Takaoka | 4 | -84/+21 |
2014-01-09 | Passing SpacingAndPunctuations to RichInputConnection.getNthPreviousWords•••Change-Id: I174c50f509ed6998b755e1a712e7f6c0f82f4425
|  Tadashi G. Takaoka | 4 | -33/+59 |
2014-01-09 | Passing SpacingAndPunctuations to CapsModeUtils.getCapsMode•••Change-Id: I0b06e8cc75a403f7061864c5b7f3f6a2cacd60eb
|  Tadashi G. Takaoka | 3 | -52/+59 |
2014-01-09 | Add American typography boolean to SpacingAndPunctuations•••Change-Id: Ic2663eaef38fbe7c5bd1fb3a81771e94c2d4de52
|  Tadashi G. Takaoka | 4 | -10/+19 |
2014-01-09 | Fix a unit test in InputLogicTests.•••Change-Id: I63a20bb2e4b3c39495fade6cae3a14fdd11ca48c
|  Keisuke Kuroyanagi | 1 | -1/+1 |
2014-01-09 | A style fix•••Just a style fix to follow up the previous commit.
BUG: 11439600
Change-Id: Ib7a4654627a6fc22d86f2097f86f001bd76426e9
|  Yohei Yukawa | 1 | -1/+1 |
2014-01-09 | Remove some warnings•••Change-Id: I5cfa0d2fccc139bd6c45c5590a68c3e0c90534b8
|  Jean Chalard | 12 | -81/+61 |
2014-01-09 | Fix a bug where changing auto-shift would cancel autorepeat.•••Bug: 12453966
Change-Id: Idd65f732e2de0a7309e08315893c28cca63732c5
|  Jean Chalard | 2 | -3/+3 |
2014-01-09 | Trigger haptic feedback at touch down on the EmojiPalettesView•••EmojiPalettesView triggers the haptic feedback at the timing of touch-up rather than touch-down.
This patch set changes the timing to touch-up so that the UX on the EmojiPalettesView can be consistent with normal keyboard layouts.
This patch set also fixes the missing haptic feedback from facemarks such as ":-)".
Bug: 11439600
Change-Id: I22245946712bd1c36226786d6ff81e3b563f7ef7
|  Yohei Yukawa | 2 | -29/+51 |
2014-01-08 | Import translations. DO NOT MERGE•••Change-Id: Ia2d0f4528529df463307e1706cd4d7265b3958c1
Auto-generated-cl: translation import
|  Baligh Uddin | 31 | -62/+31 |
2014-01-08 | Check whether the word is OOV or not for amanatto update.•••Bug: 11736529
Change-Id: I27973212da7e922b2185dbb4febfb5bb4dfbdcbb
|  Keisuke Kuroyanagi | 4 | -1/+5 |
2014-01-08 | Make reloadMainDict() private.•••Bug: 8187060
Change-Id: I852a751d748f39512e62c9cd69748dfbe87b2151
|  Keisuke Kuroyanagi | 3 | -37/+84 |
2014-01-08 | Revert "Revert "Separate spacing and punctuation related settings values""•••This reverts commit 8bc427799a9b7a53103d1edba4a447033fbd8cfa.
Change-Id: I34f509998afdb1a7c120fbfdb9f0a1b188224a4d
|  Jean Chalard | 6 | -85/+153 |
2014-01-08 | 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 | 2 | -5/+18 |
2014-01-08 | 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 | 6 | -153/+85 |
2014-01-08 | Revert "Fix a race condition, take 2."•••Conflicts prevent this to be cherry-picked.
This reverts commit dd3d697a6e6713d82020dd63cbf78e4f87664484.
Change-Id: Ib97fae2234633b4bb27d611f48a79060db9ab16f
|  Jean Chalard | 2 | -18/+5 |
2014-01-08 | 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 | 2 | -5/+18 |
2014-01-08 | Revert "Revert "Fix a race condition.""•••This reverts commit 38401272446dbaec785e656b921cfbd31162ba09.
Change-Id: I7f0d975ec9e18d94fe3505250360c2b79ccbe96d
|  Jean Chalard | 1 | -2/+5 |
2014-01-08 | Rename InputPointers.addPointer and ResizableIntArray.add•••Change-Id: I5580250c91b29f93ed886b080ce33845b3b67ace
|  Tadashi G. Takaoka | 6 | -21/+19 |
2014-01-08 | Fix ResizableIntArrayTests readability•••Change-Id: I68862b98f228210f1d76fef4fd7454edbfd8e01d
|  Tadashi G. Takaoka | 2 | -69/+90 |
2014-01-08 | 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 | 2 | -95/+163 |
2014-01-08 | Fix: NPE in LatinIME.loadSettnigs().•••Change-Id: Icae6c74401a7a8f4da6015737f09f557640c0cca
|  Keisuke Kuroyanagi | 1 | -2/+4 |
2014-01-08 | Separate spacing and punctuation related settings values•••Bug: 12179576
Change-Id: Ibafd824a0ea2251843b388e95dcbd8f9bca92d5c
|  Tadashi G. Takaoka | 6 | -85/+153 |
2014-01-07 | Revert "Fix a race condition."•••This reverts commit 42e83c7f06a1683b27a11564aeb9f3999b10382f.
Bug: 12434783
Change-Id: Id01b6d14906430b13e64f5fc4d7fb01572d0a960
|  Victoria Lease | 1 | -5/+2 |
2014-01-07 | Import translations. DO NOT MERGE•••Change-Id: Ia0ef830a5bb3ed6fee1a5d113b2baec60bb51a97
Auto-generated-cl: translation import
|  Baligh Uddin | 55 | -10/+120 |
2014-01-07 | Import translations. DO NOT MERGE•••Change-Id: I3e0560396e087669ff484b6c15d0f0eb03c02de5
Auto-generated-cl: translation import
|  Baligh Uddin | 2 | -10/+10 |
2014-01-07 | Prepare using dictionaryFacilitator for amanatto updates.•••Bug: 11736529
Change-Id: I0d9d9b1c1108b11064163df7c2089f53e92cd7ad
|  Keisuke Kuroyanagi | 2 | -4/+12 |
2014-01-07 | Update personalized dict only when single subtype is used.•••Bug: 12424891
Change-Id: If041d1ab45988dbf9b878ae3367fee0fbc8a8988
|  Keisuke Kuroyanagi | 3 | -3/+19 |
2014-01-07 | Fix a race condition.•••Don't use absolute cursor positions, this leads to race
conditions.
Bug: 12390573
Change-Id: I6f04f07a46babbc3d51cb94c7eaee21bed242396
|  Jean Chalard | 1 | -2/+5 |
2014-01-06 | Fix possible NPE.•••Bug: 11933670
Change-Id: Ib0ce853e691bdadffb73874ea9f188cfbec8bc93
|  Keisuke Kuroyanagi | 3 | -2/+19 |
2014-01-06 | Remove language switch key from symbols and symbols shift keyboard•••This change also fixes the background color of the left keys of the space key.
Bug: 12407208
Change-Id: Id869605146e9f72835f892c3c7ae14f118177394
|  Tadashi G. Takaoka | 5 | -47/+8 |
2014-01-06 | Add description of disabled voice input key option•••Bug: 12377179
Change-Id: I44d35c6fa82883710674ecca6f630d09076472fe
|  Tadashi G. Takaoka | 2 | -2/+10 |
2014-01-06 | Update shortcut IME info before checking availability of it•••Bug: 12377179
Change-Id: I4e2de78dc7e21f3f34eb56c790160b77cfc7584c
|  Tadashi G. Takaoka | 1 | -2/+6 |
2014-01-06 | Add DictionaryFacilitatorForSuggest constructor for testing•••This change must be checked in together with I00b6e79d6b.
Change-Id: Id5f87a8d0286eb44958381fefd428f16896dc0d4
|  Tadashi G. Takaoka | 3 | -11/+51 |
2013-12-27 | [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 | 5 | -80/+32 |
2013-12-27 | [IL51] Make private what can be made private•••Bug: 8636060
Change-Id: Ifa027de465a421b67897d51e9ad47666dda117ad
|  Jean Chalard | 1 | -6/+4 |
2013-12-27 | [IL50] Move the InputUpdater out of LatinIME.•••...at last
Bug: 8636060
Change-Id: If5e27a9d28ff64d318dd76792f55edfc8a78d2d7
|  Jean Chalard | 3 | -201/+168 |
2013-12-27 | [IL49] Move InputLogic-related init code to InputLogic.•••Bug: 8636060
Change-Id: Ie5027512f066d9c530a686ac559e4649a39bf439
|  Jean Chalard | 2 | -19/+22 |
2013-12-27 | Fix a failing test•••How/when did this happen? How was it not noticed?
Change-Id: I352a8dad65f8c27bf5120d567e96d9a12cfa8496
|  Jean Chalard | 1 | -2/+3 |
2013-12-27 | [IL48] Move onEndBatchInputAsyncInternal to InputLogic.•••Bug: 8636060
Change-Id: Idfabf7231a80e21786ce1910b5140bb3dddcac7d
|  Jean Chalard | 2 | -35/+43 |
2013-12-27 | [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 | 1 | -23/+10 |
2013-12-27 | [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 | 1 | -20/+2 |
2013-12-27 | [IL45] Remove lingering messages•••Bug: 8636060
And possibly bug: 12085864
Change-Id: I867036bc3d8bd255dc8ceebdd30e28b4b3dc4b2d
|  Jean Chalard | 1 | -4/+9 |
2013-12-27 | [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 | 2 | -27/+63 |
2013-12-27 | Fix possible NPE in FileUtils.•••Change-Id: I503f91e266c71e2370a5807d171e2254c334f7cb
|  Keisuke Kuroyanagi | 1 | -2/+8 |