| Commit message (Expand) | Author | Age | Files | Lines |
|---|
| * | Merge "[IL84] Remove a useless call and rename a method" |   Jean Chalard | 2014-01-27 | 1 | -2/+2 |
| |\ |
|
| | * | [IL84] Remove a useless call and rename a method•••Bug: 8636060
Change-Id: I5cac787953a3c71b6402520c2f5c70619d4a40a8
| Jean Chalard | 2014-01-27 | 1 | -2/+2 |
| * | | Merge "[IL81] Remove a useless test." | Jean Chalard | 2014-01-27 | 1 | -12/+3 |
| |\| |
|
| | * | [IL81] Remove a useless test.•••It seems the comments were right all along. This seems to really
be useless.
Bug: 8636060
Change-Id: Ie030e9cb73007334380d023abeaa4ce52a863a34
| Jean Chalard | 2014-01-27 | 1 | -12/+3 |
| * | | Merge "Try to figure out whether d.quotes open or close." | Jean Chalard | 2014-01-27 | 1 | -5/+21 |
| |\ \
| |/
|/| |
|
| | * | Try to figure out whether d.quotes open or close.•••Bug: 8911672
Change-Id: I5d5635949530a67f95e5208986907251b7bce903
| Jean Chalard | 2014-01-27 | 1 | -5/+21 |
| * | | [IL80] Reverse a test for clarity.•••Bug: 8636060
Change-Id: I5092942ab4fd6aaf37023083040cadbc18583fd1
| Jean Chalard | 2014-01-27 | 1 | -50/+49 |
| * | | [IL79] Some refactoring•••Bug: 8636060
Change-Id: I931c553f68a1f15d22711a661dbffd4e5d421979
| Jean Chalard | 2014-01-27 | 1 | -18/+10 |
| * | | [IL78] Move most of onUpdateSelection to InputLogic.•••Bug: 8636060
Change-Id: I7290250637e37bf6964bab71d2141e19f20a5459
| Jean Chalard | 2014-01-27 | 1 | -0/+79 |
| * | | [IL77] Remove refs to Settings from InputLogic•••Bug: 8636060
Change-Id: I4ce5c9b62181ce8593a453ce222ae7d1fdc08f07
| Jean Chalard | 2014-01-27 | 1 | -9/+3 |
| * | | [IL75] Move the SubtypeSwitcher out of InputLogic.•••Bug: 8636060
Change-Id: I2a5da27e73cf378e55bd80ab2054c1ba8b13f6ec
| Jean Chalard | 2014-01-27 | 1 | -5/+4 |
| |/ |
|
| * | Merge "[IL76] Remove a SuggestionStripView dependency" | Jean Chalard | 2014-01-23 | 1 | -3/+2 |
| |\ |
|
| | * | [IL76] Remove a SuggestionStripView dependency•••This constant is better located in SuggestedWords.
Or it could be in Constants, that would be fine too.
Bug: 8636060
Change-Id: I3e721bb1e2559f028dce8929eceadfe0478c5924
| Jean Chalard | 2014-01-23 | 1 | -3/+2 |
| * | | [IL66.1] Add a comment•••Followup to IL66
Change-Id: I067e72c36f483cc3e3b24736c01843531298b507
| Jean Chalard | 2014-01-23 | 1 | -1/+1 |
| |/ |
|
| * | [IL70] Remove unused arguments•••Bug: 8636060
Change-Id: Iec56bce362ebfc52fedab02b127978abb4f3c7f9
| Jean Chalard | 2014-01-23 | 1 | -11/+7 |
| * | [IL69] Oust Keyboard from WordComposer#setComposingWord•••Bug: 8636060
Change-Id: I624a45500603dfad355ee3b8b794b2d895219c0b
| Jean Chalard | 2014-01-23 | 1 | -6/+9 |
| * | [IL67] Externalize a call to the keyboard switcher•••Bug: 8636060
Change-Id: Ib13a54854e30afb79217a0a1f2aeca50a95ba155
| Jean Chalard | 2014-01-23 | 1 | -9/+6 |
| * | [IL66] Remove two accesses to KeyboardSwitcher.•••Bug: 8636060
Change-Id: I3b144993e2bcd552f9b292b8f057c9db4216125c
| Jean Chalard | 2014-01-23 | 1 | -7/+7 |
| * | [IL64] Pull up X,Y processing, step 3•••Bug: 8636060
Change-Id: Ic051e5d5514d270101b0571a2d30e2caa8f85bc1
| Jean Chalard | 2014-01-22 | 1 | -20/+4 |
| * | [IL63] Pull up X,Y processing, step 2•••This also fixes the coordinates passed to logOnDelete and
log onSeparator.
Bug: 8636060
Change-Id: I4703d61ad7de76b501adca18df3a69e06450c9a1
| Jean Chalard | 2014-01-22 | 1 | -20/+20 |
| * | [IL62] Pull up X,Y processing, step 1•••This is fine because getKey{X,Y} is idempotent for any
non-keyboard coordinate value.
However this makes a net change : the x,y values passed to
LatinImeLoggerUtils.onNonSeparator are now different.
The point is however that they used to be wrong. The logged
values used not to account for the keyboard padding that
is present on tablets, and in the simulating tools we don't
know about that padding so we couldn't use the coordinates.
The catch here is that other calls like LoggerUtils.onSeparator
should follow suit, but this is too big a change to do it at once.
Follow-up changes will fix them too.
Bug: 8636060
Change-Id: If4b3d3cb1ed4b44c35f23e66aba3b5797236bba7
| Jean Chalard | 2014-01-22 | 1 | -8/+9 |
| * | Merge "[IL61] Remove an internal use of key coordinates" | Jean Chalard | 2014-01-22 | 1 | -9/+9 |
| |\ |
|
| | * | [IL61] Remove an internal use of key coordinates•••Bug: 8636060
Change-Id: Id87b0ac4a99ec6a26db6e250d6cd22defc02a23d
| Jean Chalard | 2014-01-17 | 1 | -9/+9 |
| * | | Use sorted int[] to represent word separators•••Change-Id: I4103541d99fe59bfcf12379a1298a0a690497846
|   Tadashi G. Takaoka | 2014-01-20 | 1 | -2/+3 |
| * | | Refactor logic to retrieve relevant suggestions a bit•••Change-Id: Ic7d2cbb2c1b2deaa4e735484bdc7413c0b3b1939
| Tadashi G. Takaoka | 2014-01-20 | 1 | -1/+2 |
| * | | Refactor RichInputConnection a bit•••Change-Id: I866488a47ca04ca587e805663dfd597bb7b1ebce
| Tadashi G. Takaoka | 2014-01-17 | 1 | -4/+5 |
| |/ |
|
| * | [IL59] Rename a method•••Um... this is not async ? Remove that from the name
Bug: 8636060
Change-Id: Ida5f9bd6929eae75103eb2e3fb866ed7468af806
| Jean Chalard | 2014-01-17 | 1 | -1/+1 |
| * | [IL58] Simplification•••No need to test multiple times for this. Let's just never do useless
stuff, and only test for that once.
Bug: 8636060
Change-Id: I50a9e4da769fbec44fbb12eedfed03aad924cc2c
| Jean Chalard | 2014-01-17 | 1 | -2/+0 |
| * | [IL56] Don't erase suggestions when reverting a commit.•••Bug: 12297051
Change-Id: Iffa1c98b8fe2b816c3ac6bc171037288107f637d
| Jean Chalard | 2014-01-16 | 1 | -12/+40 |
| * | Pass time stamp to addToUserHistory method as an argument•••Bug: 12179576
Change-Id: I93037ee2aa7c0432400fe2f93fb60d2e60f70f5f
| Tadashi G. Takaoka | 2014-01-14 | 1 | -1/+7 |
| * | [IL55] Remove a useless argument.•••Change-Id: I0b3a6cdf49038c32194413c21da824784a53f581
| Jean Chalard | 2014-01-10 | 1 | -15/+2 |
| * | [IL54.5] Use the cursor pos estimate, not the last value•••For edit tasks, the estimate is actually the right thing to use.
This is really dangerous, but it will get rid of pretty much all
race conditions.
Change-Id: I2d5ca3ce45e32f1bd9c8b778421fd54b9c1f6f63
| Jean Chalard | 2014-01-10 | 1 | -80/+36 |
| * | [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 | 2014-01-10 | 1 | -0/+2 |
| * | Fix an NPE.•••Bug: 12397228
Change-Id: I7632931f0685fc8f0558946be66025b2bb2b5a3d
| Jean Chalard | 2014-01-10 | 1 | -3/+4 |
| * | Passing SpacingAndPunctuations to RichInputConnection.getNthPreviousWords•••Change-Id: I174c50f509ed6998b755e1a712e7f6c0f82f4425
| Tadashi G. Takaoka | 2014-01-09 | 1 | -9/+13 |
| * | Revert "Revert "Separate spacing and punctuation related settings values""•••This reverts commit 8bc427799a9b7a53103d1edba4a447033fbd8cfa.
Change-Id: I34f509998afdb1a7c120fbfdb9f0a1b188224a4d
| Jean Chalard | 2014-01-08 | 1 | -12/+13 |
| * | 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 | 1 | -5/+14 |
| * | Merge "Revert "Separate spacing and punctuation related settings values"" | Jean Chalard | 2014-01-08 | 1 | -13/+12 |
| |\ |
|
| | * | 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 | 1 | -13/+12 |
| * | | 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 | 1 | -14/+5 |
| * | | 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 | 1 | -5/+14 |
| * | | Revert "Revert "Fix a race condition.""•••This reverts commit 38401272446dbaec785e656b921cfbd31162ba09.
Change-Id: I7f0d975ec9e18d94fe3505250360c2b79ccbe96d
| Jean Chalard | 2014-01-08 | 1 | -2/+5 |
| |/ |
|
| * | Separate spacing and punctuation related settings values•••Bug: 12179576
Change-Id: Ibafd824a0ea2251843b388e95dcbd8f9bca92d5c
| Tadashi G. Takaoka | 2014-01-08 | 1 | -12/+13 |
| * | [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 | 1 | -38/+23 |
| * | [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 | 1 | -39/+33 |
| * | [IL49] Move InputLogic-related init code to InputLogic.•••Bug: 8636060
Change-Id: Ie5027512f066d9c530a686ac559e4649a39bf439
| Jean Chalard | 2013-12-27 | 1 | -2/+19 |
| * | [IL48] Move onEndBatchInputAsyncInternal to InputLogic.•••Bug: 8636060
Change-Id: Idfabf7231a80e21786ce1910b5140bb3dddcac7d
| Jean Chalard | 2013-12-27 | 1 | -0/+41 |
| * | [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 | 1 | -1/+1 |
| * | Fix a bug where predictions would not pop upon manual pick•••Bug: 12295276
Change-Id: Id359c92acde44758b12929e7bac719d5c9c7577b
| Jean Chalard | 2013-12-26 | 1 | -2/+5 |
