aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin/inputlogic (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
* | [IL99] Migrate some code to InputLogic.Jean Chalard2014-01-281-0/+19
|/ | | | | Bug: 8636060 Change-Id: Ibf1f9c7bdc3fdd872d6c15447fa555434485cc52
* Merge "[IL86] Simplification and correctness fix."Jean Chalard2014-01-271-5/+1
|\
| * [IL86] Simplification and correctness fix.Jean Chalard2014-01-271-5/+1
| | | | | | | | | | | | | | This should be what was intended. Bug: 8636060 Change-Id: I20a1cc4f0b1fb9444fb6f9a20ef727e3d1a3a880
* | Merge "[IL84] Remove a useless call and rename a method"Jean Chalard2014-01-271-2/+2
|\|
| * [IL84] Remove a useless call and rename a methodJean Chalard2014-01-271-2/+2
| | | | | | | | | | Bug: 8636060 Change-Id: I5cac787953a3c71b6402520c2f5c70619d4a40a8
* | Merge "[IL81] Remove a useless test."Jean Chalard2014-01-271-12/+3
|\|
| * [IL81] Remove a useless test.Jean Chalard2014-01-271-12/+3
| | | | | | | | | | | | | | | | It seems the comments were right all along. This seems to really be useless. Bug: 8636060 Change-Id: Ie030e9cb73007334380d023abeaa4ce52a863a34
* | Merge "Try to figure out whether d.quotes open or close."Jean Chalard2014-01-271-5/+21
|\ \ | |/ |/|
| * Try to figure out whether d.quotes open or close.Jean Chalard2014-01-271-5/+21
| | | | | | | | | | Bug: 8911672 Change-Id: I5d5635949530a67f95e5208986907251b7bce903
* | [IL80] Reverse a test for clarity.Jean Chalard2014-01-271-50/+49
| | | | | | | | | | Bug: 8636060 Change-Id: I5092942ab4fd6aaf37023083040cadbc18583fd1
* | [IL79] Some refactoringJean Chalard2014-01-271-18/+10
| | | | | | | | | | Bug: 8636060 Change-Id: I931c553f68a1f15d22711a661dbffd4e5d421979
* | [IL78] Move most of onUpdateSelection to InputLogic.Jean Chalard2014-01-271-0/+79
| | | | | | | | | | Bug: 8636060 Change-Id: I7290250637e37bf6964bab71d2141e19f20a5459
* | [IL77] Remove refs to Settings from InputLogicJean Chalard2014-01-271-9/+3
| | | | | | | | | | Bug: 8636060 Change-Id: I4ce5c9b62181ce8593a453ce222ae7d1fdc08f07
* | [IL75] Move the SubtypeSwitcher out of InputLogic.Jean Chalard2014-01-271-5/+4
|/ | | | | Bug: 8636060 Change-Id: I2a5da27e73cf378e55bd80ab2054c1ba8b13f6ec
* Merge "[IL76] Remove a SuggestionStripView dependency"Jean Chalard2014-01-231-3/+2
|\
| * [IL76] Remove a SuggestionStripView dependencyJean Chalard2014-01-231-3/+2
| | | | | | | | | | | | | | | | This constant is better located in SuggestedWords. Or it could be in Constants, that would be fine too. Bug: 8636060 Change-Id: I3e721bb1e2559f028dce8929eceadfe0478c5924
* | [IL66.1] Add a commentJean Chalard2014-01-231-1/+1
|/ | | | | | Followup to IL66 Change-Id: I067e72c36f483cc3e3b24736c01843531298b507
* [IL70] Remove unused argumentsJean Chalard2014-01-231-11/+7
| | | | | Bug: 8636060 Change-Id: Iec56bce362ebfc52fedab02b127978abb4f3c7f9
* [IL69] Oust Keyboard from WordComposer#setComposingWordJean Chalard2014-01-231-6/+9
| | | | | Bug: 8636060 Change-Id: I624a45500603dfad355ee3b8b794b2d895219c0b
* [IL67] Externalize a call to the keyboard switcherJean Chalard2014-01-231-9/+6
| | | | | Bug: 8636060 Change-Id: Ib13a54854e30afb79217a0a1f2aeca50a95ba155
* [IL66] Remove two accesses to KeyboardSwitcher.Jean Chalard2014-01-231-7/+7
| | | | | Bug: 8636060 Change-Id: I3b144993e2bcd552f9b292b8f057c9db4216125c
* [IL64] Pull up X,Y processing, step 3Jean Chalard2014-01-221-20/+4
| | | | | Bug: 8636060 Change-Id: Ic051e5d5514d270101b0571a2d30e2caa8f85bc1
* [IL63] Pull up X,Y processing, step 2Jean Chalard2014-01-221-20/+20
| | | | | | | | This also fixes the coordinates passed to logOnDelete and log onSeparator. Bug: 8636060 Change-Id: I4703d61ad7de76b501adca18df3a69e06450c9a1
* [IL62] Pull up X,Y processing, step 1Jean Chalard2014-01-221-8/+9
| | | | | | | | | | | | | | | | | | 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
* Merge "[IL61] Remove an internal use of key coordinates"Jean Chalard2014-01-221-9/+9
|\
| * [IL61] Remove an internal use of key coordinatesJean Chalard2014-01-171-9/+9
| | | | | | | | | | Bug: 8636060 Change-Id: Id87b0ac4a99ec6a26db6e250d6cd22defc02a23d
* | Use sorted int[] to represent word separatorsTadashi G. Takaoka2014-01-201-2/+3
| | | | | | | | Change-Id: I4103541d99fe59bfcf12379a1298a0a690497846
* | Refactor logic to retrieve relevant suggestions a bitTadashi G. Takaoka2014-01-201-1/+2
| | | | | | | | Change-Id: Ic7d2cbb2c1b2deaa4e735484bdc7413c0b3b1939
* | Refactor RichInputConnection a bitTadashi G. Takaoka2014-01-171-4/+5
|/ | | | Change-Id: I866488a47ca04ca587e805663dfd597bb7b1ebce
* [IL60] Make InputLogicHandler package privateJean Chalard2014-01-171-2/+1
| | | | | | | Yay Bug: 8636060 Change-Id: Iad9c3804504635094e861999198f4baf70e61955
* [IL59] Rename a methodJean Chalard2014-01-171-1/+1
| | | | | | | Um... this is not async ? Remove that from the name Bug: 8636060 Change-Id: Ida5f9bd6929eae75103eb2e3fb866ed7468af806
* [IL58] SimplificationJean Chalard2014-01-171-2/+0
| | | | | | | | 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
* [IL56] Don't erase suggestions when reverting a commit.Jean Chalard2014-01-161-12/+40
| | | | | Bug: 12297051 Change-Id: Iffa1c98b8fe2b816c3ac6bc171037288107f637d
* Pass time stamp to addToUserHistory method as an argumentTadashi G. Takaoka2014-01-141-1/+7
| | | | | Bug: 12179576 Change-Id: I93037ee2aa7c0432400fe2f93fb60d2e60f70f5f
* [IL55] Remove a useless argument.Jean Chalard2014-01-101-15/+2
| | | | Change-Id: I0b3a6cdf49038c32194413c21da824784a53f581
* [IL54.5] Use the cursor pos estimate, not the last valueJean Chalard2014-01-101-80/+36
| | | | | | | | 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
* [IL54] Add a mirror method to fix the cursor pos estimateJean Chalard2014-01-101-0/+2
| | | | | | | This just mirrors what InputLogic#tryFixLyingCursorPosition is doing. That method will go away in the next change. Change-Id: Ifa2827dbc1f1d20e2c642d6f2d23514a01ed9203
* Fix an NPE.Jean Chalard2014-01-102-3/+31
| | | | | Bug: 12397228 Change-Id: I7632931f0685fc8f0558946be66025b2bb2b5a3d
* Passing SpacingAndPunctuations to RichInputConnection.getNthPreviousWordsTadashi G. Takaoka2014-01-091-9/+13
| | | | Change-Id: I174c50f509ed6998b755e1a712e7f6c0f82f4425
* Revert "Revert "Separate spacing and punctuation related settings values""Jean Chalard2014-01-081-12/+13
| | | | | | This reverts commit 8bc427799a9b7a53103d1edba4a447033fbd8cfa. Change-Id: I34f509998afdb1a7c120fbfdb9f0a1b188224a4d
* Fix a race condition, take 2.Jean Chalard2014-01-081-5/+14
| | | | | | | | | | 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
* Merge "Revert "Separate spacing and punctuation related settings values""Jean Chalard2014-01-081-13/+12
|\
| * Revert "Separate spacing and punctuation related settings values"Jean Chalard2014-01-081-13/+12
| | | | | | | | | | | | | | | | This is conflicting with later changes. Temporary revert for cherry-pick. This reverts commit 0b593ce858d4b406e949f0e809cb77f41da691f9. Change-Id: Id53eadb023a950cfcca496c0cfbfe583c7ec7b8c
* | Revert "Fix a race condition, take 2."Jean Chalard2014-01-081-14/+5
| | | | | | | | | | | | | | | | Conflicts prevent this to be cherry-picked. This reverts commit dd3d697a6e6713d82020dd63cbf78e4f87664484. Change-Id: Ib97fae2234633b4bb27d611f48a79060db9ab16f
* | Fix a race condition, take 2.Jean Chalard2014-01-081-5/+14
| | | | | | | | | | | | | | | | | | | | 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
* | Revert "Revert "Fix a race condition.""Jean Chalard2014-01-081-2/+5
|/ | | | | | This reverts commit 38401272446dbaec785e656b921cfbd31162ba09. Change-Id: I7f0d975ec9e18d94fe3505250360c2b79ccbe96d
* Separate spacing and punctuation related settings valuesTadashi G. Takaoka2014-01-081-12/+13
| | | | | Bug: 12179576 Change-Id: Ibafd824a0ea2251843b388e95dcbd8f9bca92d5c
* [IL52] Remove a useless method.Jean Chalard2013-12-271-38/+23
| | | | | | | | | | | | | | | 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
* [IL51] Make private what can be made privateJean Chalard2013-12-271-6/+4
| | | | | Bug: 8636060 Change-Id: Ifa027de465a421b67897d51e9ad47666dda117ad
* [IL50] Move the InputUpdater out of LatinIME.Jean Chalard2013-12-272-40/+157
| | | | | | | ...at last Bug: 8636060 Change-Id: If5e27a9d28ff64d318dd76792f55edfc8a78d2d7