aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin/RichInputConnection.java (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge "Move util classes to the latin/utils directory"Ken Wakasa2013-06-241-2/+5
|\ \
| * | Move util classes to the latin/utils directoryKen Wakasa2013-06-241-2/+5
| |/ | | | | | | Change-Id: I1c5b27c8edf231680edb8d96f63b9d04cfc6a6fa
* / Refactor text range to be able to get spans larger than the wordJean Chalard2013-06-241-19/+24
|/ | | | | | | | | This changes how the Range class stores its data, but not its functionality. It also improves encapsulation a bit. Bug: 8839763 Bug: 8862327 Change-Id: I5bd583b3fc96a99b93a2632882d8fd587c03ab76
* Optimizations and safeguards.Jean Chalard2013-05-081-2/+3
| | | | | | | | | | | - Don't call finishComposingText when useless. - Add safeguards against calling setComposingRegion when the data returned by the editor is inconsistent. - Cancel pending recorrection messages when new messages arrive. Bug: 8842941 Bug: 8845001 Change-Id: I939701033cf5c2bbd85871ecf83e329021ddeb91
* Don't insert automatic spaces when text looks like a URLJean Chalard2013-04-161-1/+11
| | | | | | | | | | | | | | | This is about as ad-hoc as it gets, but then again, what we want is probably as ad-hoc as it gets. All URL boxes I know of double as search bars, and not adding automatic spaces there sucks (e.g. in Chrome URL bar). And in other boxes actually you don't want to add a space if it looks like a URL. QSB isn't even a search box, and it behaves like this. So I think this is actually the right answer to the problem. Bug: 7062925 Change-Id: Ib09472b34644fd5bf2dc84bb97cedeeba28bcd02
* Have Latin IME re-capitalize a selected stringJean Chalard2013-04-151-0/+5
| | | | | | | | | | Upon pressing Shift, if there is currently a selected string, have Latin IME change its capitalization. This does not yet have the keyboard mode follow the mode - the change is complicated enough as is. Bug: 7657025 Change-Id: I54fe8485f44e04efd72c71ac9feee5ce21ba06f2
* Merge "Restart suggestions when the cursor moves."Jean Chalard2013-04-121-7/+12
|\
| * Restart suggestions when the cursor moves.Jean Chalard2013-04-121-7/+12
| | | | | | | | | | | | | | | | | | | | | | This uses the old suggestions. It does not try to recompute new suggestions if there are no old suggestions yet: this is coming in a later change. If there are no suggestions, this shows the word itself as a suggestion. Bug: 8084810 Change-Id: I4c2e25df0ff3673be1825f57a0c19a9d23d47a48
* | Fix a bug with bad application completionsJean Chalard2013-04-121-1/+3
|/ | | | | Bug: 8561056 Change-Id: I91fb0ac76dabfc841bafc0e2c0ea9f63e6f604f9
* Fix an NPEJean Chalard2013-03-291-18/+10
| | | | | | | | Also remove an unused method and make 2 private variables final. Bug: 8461093 Change-Id: I676d93fda28e82eb5a6978f2e7ad5a0add2cbecf
* Rename ProductionFlag.IS_EXPERIMENTAL to USES_DEVELOPMENT_ONLY_DIAGNOSTICSKen Wakasa2013-03-181-10/+10
| | | | | bug: 8393568 Change-Id: Ie5edf44a3627aca9416145aff56bf05bbf2a05f3
* Separate caps mode related utils to CapsModeUtilsTadashi G. Takaoka2013-02-121-1/+1
| | | | Change-Id: I32700c434b296bb3fd39e040c2dda1fe90667daf
* Fix Apache license commentTadashi G. Takaoka2013-01-211-8/+8
| | | | Change-Id: Ic56167f952a7f4449da366e1e81610e72c966086
* Simplify the space-before and space-after logic.Jean Chalard2013-01-171-8/+5
| | | | | | Bug: 7889078 Bug: 7268000 Change-Id: I6c77b8c9e60ef69f02526b407124d2f5d02818ee
* [Rlog50] capture bigrams properly even with deletionsKurt Partridge2013-01-091-4/+5
| | | | | | multi-project commit with Ia4ec213e8356897807cb6a278fccdbaa945732f0 Change-Id: Ib3fe886dc889954a31586ab81d00a21d8d55efd2
* Merge "[Rlog32] ResearchLogger segment motion data around gestures correctly"Kurt Partridge2013-01-081-3/+3
|\
| * [Rlog32] ResearchLogger segment motion data around gestures correctlyKurt Partridge2013-01-071-3/+3
| | | | | | | | | | | | multi-project commit with I684ba5fed8c653777942e6b0366113ca40dfa37c Change-Id: Id3804e6cfa97b4cdde212b8eebc413f76fa4772c
* | Small code cleanupsKen Wakasa2013-01-071-28/+28
| | | | | | | | | | | | Multi-project commit with I249d5fbe Change-Id: Ia28c4e970992aa1299a30e604eaa5d096655c3a5
* | Rename some confusing variablesJean Chalard2012-12-271-12/+12
| | | | | | | | Change-Id: Ib0de800599ae7f12c86270a627616d5b52366414
* | Fix an old bug in getting a word from the connected view.Jean Chalard2012-12-271-8/+3
|/ | | | | | | | | | | | | | In this test, it's impossible that start < 0 so the test is useless. I'm not sure what the cursor test was for, but it's very old code, and it seems the assumption was either misled or doesn't hold any more: testing for the absolute cursor position against the length of the word against the cursor makes no sense. The net result of this was that when the cursor index got large enough, resuming suggestion would not work any more. Bug: 7586467 Change-Id: I3462082374fe9579bec7698f4d424de6ff5f2ded
* [Rlog24] ResearchLogger detect word boundaries in callers above commitTextKurt Partridge2012-12-231-5/+10
| | | | | | multi project commit with I5847d4aba97a7224ae170a7153811c5a11816449 Change-Id: Ie6fbf6f19f0193451fe6f3ddf9d4ef1637a35db1
* Merge "[Rlog2] ResearchLogging fix multi-space logging"Kurt Partridge2012-12-171-3/+0
|\
| * [Rlog2] ResearchLogging fix multi-space loggingKurt Partridge2012-12-171-3/+0
| | | | | | | | | | | | | | | | | | multi-space logging should look like single-space logging, missing a few minor log statements (SuggestionUpdates, SetComposingText) multi-project commit with I2af842348c2f2b8f7271ac5b63def245e83df24d Change-Id: Icd3187c0d0377255f82787afffea657c14345803
* | Privatize a few constants in BinaryDictionary.javaKen Wakasa2012-12-171-1/+1
| | | | | | | | Change-Id: I7defaf1f577fd67e678cac83ff935e8181dd0a48
* | resolved conflicts for merge of d48d6fee to masterKen Wakasa2012-12-151-0/+18
|\ \ | |/ |/| | | Change-Id: I21141956135debe737453dba87617671b1716d0a
| * Use the amended user dictionary word for insertionJean Chalard2012-12-141-0/+18
| | | | | | | | | | | | | | | | | | When the user edits a word before adding it to the user dictionary, the keyboard should replace whatever was committed before with the amended version. Bug: 7725834 Change-Id: I1a417be6c5a86d6a96bc2c76aca314ad8f1202a9
* | Remove a couple Eclipse and Android Lint warningsJean Chalard2012-12-061-7/+1
| | | | | | | | Change-Id: I0c29c5d2abcbf80759b996d34b534deb083cd7d3
* | am 8c470f16: am 5dee6e69: am 187e683a: Merge "Fix a bug with some text ↵Jean Chalard2012-11-161-1/+2
|\| | | | | | | | | | | | | fields" into jb-mr1.1-dev * commit '8c470f16db291e257ac538bd0ac2a9354d34010c': Fix a bug with some text fields
| * Fix a bug with some text fieldsJean Chalard2012-11-141-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Not sure when this happens exactly, but it is possible that InputConnection#getTextBeforeCursor returns null. This happens for example upon rotating the screen with the composing field empty in Gmail. In this case, StringBuilder#append will convert the null pointer into the string "null", which is sure better than a crash, but can have a number of bad side-effects, like auto-caps not working. Bug: 7533034 Change-Id: Ia1cfab432c13a12ff1c2f013c59bac05a587f553
* | Move code point constants from Keyboard to Constants classTadashi G. Takaoka2012-10-291-4/+3
| | | | | | | | Change-Id: Iee01d4d2b916d0b584531104ac865ae6e6370a3d
* | Replace useless CharSequence to StringTadashi G. Takaoka2012-10-221-13/+17
|/ | | | Change-Id: Idc478f901185ee1b4912acc82d0cbc54fee4e991
* Have "no whitespace before cursor" upon gesture trigger phantom spaceJean Chalard2012-10-171-0/+7
| | | | | | | Bug: 7359291 Bug: 7357758 Bug: 7197651 Change-Id: Ia805a87e922739ae0a06978a3bf00d91c94b6c51
* Remove debug prints.Jean Chalard2012-10-011-12/+13
| | | | | Bug: 7226098 Change-Id: Ibacb740022a6af56c9d555d6797dd402a89f6ca4
* Cleanup: Make some classes as finalTadashi G. Takaoka2012-09-271-2/+2
| | | | Change-Id: I6009b3c1950ba32b7f1e205a3db2307fe0cd688e
* Support the gesture + phantom space case (A13)Jean Chalard2012-09-181-1/+10
| | | | | Bug: 6950087 Change-Id: I198699f6e687ac74c35b9edfe469dce880d9ecaf
* Take space state into account for caps (A11)Jean Chalard2012-09-181-2/+19
| | | | | Bug: 6950087 Change-Id: I8a1bca24db64b4dd54db8ac74d90cf43cbdddab6
* Take locale into account for caps (A10)Jean Chalard2012-09-141-2/+3
| | | | | Bug: 4967874 Change-Id: Ic7ce7b2de088308fa00865c81246c84c605db1e5
* Merge "Simplify a call, and add comments (A2)" into jb-mr1-devJean Chalard2012-09-131-2/+1
|\
| * Simplify a call, and add comments (A2)Jean Chalard2012-09-131-2/+1
| | | | | | | | | | | | | | | | | | | | Since the function has to be modified heavily but does a lot of non-trivial work, add a wealth of comments explaining what it does and why so as to facilitate understanding the changes to come. Bug: 4967874 Change-Id: I6c21aea15f161d807035f279dfb7d1b98b3e9144
* | Merge "Import TextUtils.getCapsMode to fix it internally (A1)" into jb-mr1-devJean Chalard2012-09-131-1/+14
|\|
| * Import TextUtils.getCapsMode to fix it internally (A1)Jean Chalard2012-09-131-1/+14
| | | | | | | | | | | | | | | | | | This should have on effect at all on behavior, except an increase in performance. Bug: 4967874 Bug: 6950087 Change-Id: Ie2b51efefe84ca767f5dc8e3b80bfef7e1faab3d
* | Make onUpdateSelection much more resilient to race conditions.Jean Chalard2012-09-131-0/+30
|/ | | | | | | | | This is pretty much as strong as it gets. It should be impossible to get false positives and nearly impossible to get true negatives with this new code. Bug: 6981089 Change-Id: Ia32ab62f89c5943f0be169b979abab652e67bf5b
* Make Latin IME aware of its surrounding text.Jean Chalard2012-09-121-1/+178
| | | | | | | | | This is a preparatory change for Bug: 4967874 Bug: 6617760 Bug: 6950087 Change-Id: I3abf8e45c0d02c42491421f108370220134b9602
* ResearchLogger log RichInputConnectionKurt Partridge2012-08-211-26/+60
| | | | | | | | | | | | | | Move many ResearchLogger data collection points to RichInputConnection. By collecting data here, developers do not have to remember to keep the ResearchLog consistent with calls to the RichInputConnection. In addition, some unnecessary log points were removed, and the ResearchLogger is now independent of LatinImeLogger. multi-project change with I05496cfd762e9a41c42c50099143f1efc0a3d8a2 Bug: 6188932 Change-Id: I424abb2e437901262a9620255493916b7c3ff74b
* Revert "ResearchLogger log RichInputConnection"Kurt Partridge2012-08-201-60/+26
| | | | | | This reverts commit 1964ffb3f43f0ada429d13b41bd22b90a97fa9fc Change-Id: Ie43e7ac828faa91e1824a255bb3ecf6c8222d4e8
* ResearchLogger log RichInputConnectionKurt Partridge2012-08-171-26/+60
| | | | | | | | | | | | | | | | Move many ResearchLogger data collection points to RichInputConnection. By collecting data here, developers do not have to remember to keep the ResearchLog consistent with calls to the RichInputConnection. In addition, some unnecessary log points were removed, and the ResearchLogger is now independent of LatinImeLogger. multi-project change with Ib71f841044ec1072610ab5638a5edfce29b7c05b DO NOT MERGE Bug: 6188932 Change-Id: I9ec7500a5b18bfe1a5a5bb1e5bf21c43351fc59e
* ResearchLog refactorKurt Partridge2012-07-201-0/+1
| | | | | | | | | | - new package: com.android.inputmethod.research multi-project commit with Ic0a5744f3160d13218addd589890623c0d120ffc Bug: 6188932 Change-Id: Icf8d4a40a5725401799be6e209a640d99a5f34c4
* Consolidate methods that do the same thing (A74)Jean Chalard2012-07-091-46/+24
| | | | Change-Id: I6e7c3aba2b40ecefa483c9680978bc806368e96e
* Fix a bug where the caps mode would not be changedJean Chalard2012-07-061-3/+14
| | | | | Bug: 6766059 Change-Id: I378f9d35c4904c4f373260bda5863235d833eb31
* Refactoring and groundwork to fix a bug with older appsJean Chalard2012-06-251-7/+9
| | | | | | | | | | | | | | | | | | | This has a good, although small, impact on performance : it removes a two-way IPC call in a most frequent case, while possibly adding one in a rather unfrequent and less critical case. Also, this fixes a bug with surrogate pairs. This specific branch of code now correctly handles surrogate pairs. Aside from this, it should have no impact on behavior. However, since it does delay access to the previous character in the text view by a two-way IPC call, it actually goes a long way toward fixing bug#6668226. It is not really a fix and the race condition still exists, but this change makes it much, much harder to hit. Bug: 6668226 Change-Id: Id11cc6a0b7488d6bd392227cafdcf3a8d4c62f6c