| Commit message (Expand) | Author | Age | Files | Lines |
* | [IL108] Remove some uses of LatinIME in input logic.•••Access the suggestion strip through the interface instead.
Bug: 8636060
Change-Id: Ia8bd705c230f943e309d677ef683b1a2cbe68036
|  Jean Chalard | 2014-02-05 | 1 | -7/+11 |
* | Check isEligibleForAutoCommit when batch input is ended.•••Bug: 11951198
Change-Id: Ibbe09fa38eaa3ebf8f9b69f744a29db7ac598975
|  Keisuke Kuroyanagi | 2014-02-05 | 1 | -1/+4 |
* | Don't resume suggestions if suggestions are not requested.•••Bug: 12832614
Change-Id: I6a9b6d128d7e0324ba59a43cf153a3b54e1d35bf
|  Jean Chalard | 2014-02-03 | 1 | -0/+2 |
* | [IL106] Inline a now useless method.•••Bug: 8636060
Change-Id: Ic8e0225206358680aa0de421f17ab443c888036d
|  Jean Chalard | 2014-01-29 | 1 | -3/+4 |
* | [IL105] Remove more useless arguments•••Bug: 8636060
Change-Id: I9e817bb9622c84544a5ca8708e5c1fccad1ed447
|  Jean Chalard | 2014-01-29 | 1 | -3/+2 |
* | [IL104] Remove useless arguments•••Bug: 8636060
Change-Id: I97a371fe0c6227eb0f03d3e391a33726e6280322
|  Jean Chalard | 2014-01-29 | 1 | -5/+2 |
* | [IL103] Save the typed word separately in SuggestedWords.•••This helps managing the cases where the typed word is not
in the suggestions. This happens during recorrection.
Bug: 8636060
Change-Id: I6784feb793cae96272a7f1d123a0e3bbb8f03143
|  Jean Chalard | 2014-01-29 | 1 | -3/+2 |
* | Merge "Fix inconsistent behavior with the back-to-the-main-keyboard key" |  Yohei Yukawa | 2014-01-28 | 1 | -0/+4 |
|\ |
|
| * | Fix inconsistent behavior with the back-to-the-main-keyboard key•••With this patch, the back-to-the-main-keyboard in the Emoji
palette will be registered as a key-release action instead of
a key-press action, like switch-to-the-emoji-palette in the
main layout. This provides mroe consistent UX when the layout is
switched from the main layout to the Emoji palette then
switched back to the main layout.
BUG: 12464067
Change-Id: Ia0d0185db43234dfcfb7cee2677f3d199fe6ed96
|  Yohei Yukawa | 2014-01-28 | 1 | -0/+4 |
* | | [IL102] Rename a method•••Bug: 8636060
Change-Id: Ia54afdfab9124aa5c35699e357e63eae66a7557f
|  Jean Chalard | 2014-01-28 | 1 | -3/+2 |
* | | [IL101] Remove a method•••Bug: 8636060
Change-Id: Ic9c12fc6970987b0e1912e8ae85bed55260d66b5
|  Jean Chalard | 2014-01-28 | 1 | -3/+6 |
* | | [IL99] Migrate some code to InputLogic.•••Bug: 8636060
Change-Id: Ibf1f9c7bdc3fdd872d6c15447fa555434485cc52
|  Jean Chalard | 2014-01-28 | 1 | -0/+19 |
|/ |
|
* | Merge "[IL86] Simplification and correctness fix." |  Jean Chalard | 2014-01-27 | 1 | -5/+1 |
|\ |
|
| * | [IL86] Simplification and correctness fix.•••This should be what was intended.
Bug: 8636060
Change-Id: I20a1cc4f0b1fb9444fb6f9a20ef727e3d1a3a880
|  Jean Chalard | 2014-01-27 | 1 | -5/+1 |
* | | 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 |