aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin/RichInputConnection.java (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Fix the NPE on selection string on unlearnWordMohammadinamul Sheik2015-04-131-0/+2
| | | | | Bug: 20150386 Change-Id: I2ed69aba7b1fdaba1e2ac07b8218a265dfe23e09
* Extract fewer characters in getWordRangeAtCursor().Tom Ouyang2015-04-091-2/+2
| | | | | | | | | | Use LOOKBACK_CHARACTER_NUM = 80 instead of the previous EDITOR_CONTENTS_CACHE_SIZE = 1024 (which was overkill). This speeds up many InputLogic operations. Bug: 19987461 Change-Id: I62b6a589f87e5daab33376b3e48f1c615a66dcfb
* Read less data from the input connection.Dan Zivkovic2015-04-071-4/+4
| | | | | | | | | | Currently, we read 256 (max word size) * 5 (max N-gram size + 1) characters from the input connection when building our context. This is overkill. We don't need more than 80 characters, regardless of which decoder we use. Bug 19987461. Change-Id: Ie3a321cf2482adbacd8006d9d86e6601097c15ed
* Ignore the Emoji key when not connected.Dan Zivkovic2015-02-261-24/+32
| | | | | | | | | When the LatinIME does not have an active InputConnection, it will not try to toggle the Emoji keyboard. Bug 19513415. Change-Id: I31f928cd7db1cddd771c548cd3dc42f8af64d0e2
* Fix master build.Dan Zivkovic2015-02-111-1/+1
| | | | | | For some reason, we can't import android.os package in our Constants. Change-Id: I357ed72d13c5039a83c6e2489cdf5ef74d0ffea8
* Remove shortcut support from LatinIME.Dan Zivkovic2015-02-111-1/+1
| | | | | | Note this change does not affect the native decoder interface. Change-Id: I73b7dc008a5acaf75a31a36a2d332b5afabd82d0
* Move decoder specific constants to DecoderSpecificConstants.javaMohammadinamul Sheik2015-02-051-3/+6
| | | | Change-Id: Ie4d325b3152e1e7e424b8b436e222e194e4d9da0
* Removing code from RichInputConnection.Dan Zivkovic2015-02-041-49/+4
| | | | | | Followup CL that removes some more unused methods and variables. Change-Id: I4163c7cd017f59d1fd445adb6294fc89dcaafe6e
* Selections spans should not split surrogate pair.Dan Zivkovic2015-02-041-1/+24
| | | | | | | | | | When committing a span after a revert, the offset logic was such that it split a surrogate unicode pair used to express an emoji. Checking the last character of the span lets us avoid this problem. Fix for bug 19255233. Change-Id: I07d18d9002b5075f7925319dd05962011656c311
* Remnant of green [+] feature.Dan Zivkovic2015-02-041-38/+5
| | | | Change-Id: I059b1062e1d73b2fa439d9d4ee04ff0182795335
* Move StringUtils under common.Jean Chalard2014-10-281-1/+1
| | | | | Bug: 18108776 Change-Id: Ia46a4102a0e86e71118ca5e641f9f531998e166b
* Move Constants.java to the latin.common packageKen Wakasa2014-10-231-0/+1
| | | | | | | Our intention is to have classes of latinime-common under the common package as much as we can. Change-Id: I76efbbbe7bebf1a4aa943715cdff64f91675e20d
* Fix a concurrency bug that ends up in a crash.Jean Chalard2014-10-231-1/+3
| | | | | Bug: 18085768 Change-Id: I016bec997787839526ddfc521ebb99d0c7b05189
* Fix Javadoc and null analysis related warningsTadashi G. Takaoka2014-10-231-0/+3
| | | | | | | This CL also adds @SuppressWarning("unused" to java-overridable package. Bug: 18003991 Change-Id: If70527e30654384705d7a814f5efd181d9f539e1
* Fix some compiler warningsTadashi G. Takaoka2014-10-211-13/+6
| | | | | | | | | | | | | | | | | | | | | This CL fixes the following compiler warnings. - Indirect access to static member - Access to a non-accessible member of an enclosing type - Parameter assignment - Method can be static - Local variable declaration hides another field or variable - Value of local variable is not used - Unused import - Unused private member - Unnecessary 'else' statement - Unnecessary declaration of throw exception - Redundant type arguments - Missing '@Override' annotation - Unused '@SuppressWarning' annotations Bug: 18003991 Change-Id: Icfebe753e53a2cc621848f769d6a3d7ce501ebc7
* Fix an NPE.Jean Chalard2014-10-141-1/+2
| | | | | Bug: 17958289 Change-Id: I5c9ea668ff75b38c7ddebd767c36a950835c0c9f
* Add calls to stub for API to consume gesture data.David Faden2014-10-031-1/+12
| | | | | Bug: 17400259 Change-Id: Ib3511afffe1d14662e7dd14611f384689516e664
* Make a better effort to detect a framework lie.Jean Chalard2014-10-011-3/+17
| | | | | Bug: 17130496 Change-Id: I1a3631670c152d9b7667c9c4e08e14c48569eef5
* Rename PrevWordsInfo to NgramContext.Keisuke Kuroyanagi2014-09-291-4/+4
| | | | | Bug: 14425059 Change-Id: Id06a71681fa8b5e589e29fba10fe5c1cfed66984
* resolved conflicts for merge of 233a2f21 to masterYohei Yukawa2014-09-181-3/+6
|\ | | | | | | Change-Id: If391cc622367dfb4448c6a5c32b82111d352d86e
| * Use Add-To-Dictionary indicator onlyYohei Yukawa2014-09-181-3/+6
| | | | | | | | | | | | | | | | | | | | With this CL, the previously used commit indicator was reverted. Instead we use the add-to-dictionary indicator only at the moment. This CL also fixes the indicator position in bidi context. BUG: 17335734 Change-Id: I5f7cf173ddc30876e2b01320acaff8ba4265edf6
* | am 61e7afa6: am 0eaa25e0: am 914078fd: Fix a bug where recorrection would ↵Jean Chalard2014-09-081-6/+14
|\| | | | | | | | | | | | | stop on connectors * commit '61e7afa6fa98939f9dcb9f7a2ebb5678a51d4201': Fix a bug where recorrection would stop on connectors
| * Fix a bug where recorrection would stop on connectorsJean Chalard2014-09-081-6/+14
| | | | | | | | | | Bug: 16733686 Change-Id: I7a9f79a81e33a1f5bf5f3daf0b78d0f1e4447e7a
* | Fix the sentence separator for Hindi.Jean Chalard2014-09-081-4/+6
|/ | | | | Change-Id: I623e4ccbc324751eb67ec4bb777e2be5ae2a60d1 Bug: 17418371
* Follow up API signature changeYohei Yukawa2014-09-031-3/+3
| | | | | | | | This is a follow up CL for API signature change in I772c48ff18918e48a81e807b48ff907614485c09 BUG: 17320996 Change-Id: Ic8b6162bda12bf74fae79af212c5d81c400eb9e8
* Make sure to obtain the IC before requesting CursorAnchorInfoYohei Yukawa2014-08-271-2/+8
| | | | | | | | | RichInputConnection#requestUpdateCursorAnchorInfo must make sure to obtain the input connection before calling methods of it. BUG: 17299587 Change-Id: I8e0cd473a4cc32583cd47634c227d702f7c69c6c
* Set the text bgcolor only when CursorAnchorInfo is availableYohei Yukawa2014-08-271-0/+31
| | | | | | | | | | | | | When CursorAnchorInfo is unavailable, we shouldn't try to show the commit indicator and set the text highlight color. With this CL, RichInputConnection can be used to track if the application responded that it does support CursorAnchorInfo or not. This result will be taken into consideration when InputLogic needs to determine whether the commit indicator should be displayed or not. Change-Id: I945d70eeb02a7a5f3d9b22459b23d7028508910f
* Allow to add/remove background color to/from the committed textYohei Yukawa2014-08-241-2/+75
| | | | | | | | | | | | | | This is a groundwork for subsequent CLs where we need to add/remove background color to/from the commited text. In this CL, we use Spanned#SPAN_COMPOSING so that we can easily remove such a background color by calling InputConnection#finishComposingText. To make this operation easy and realiable, we need to track whether we have specified the background color to the commited text or not at one place. Here we use RichInputConnection for this purpose. Change-Id: I5f9bc4425c5d1b80a719a20e5baf336729ec08d2
* [HS3] Fix a bug on ICSJean Chalard2014-08-061-0/+22
| | | | | | | | | | There is a bug in ICS where the input connection won't take any writing commands after rotation until the cursor moves. This fixes it by wiggling the cursor position once before trying to do anything. Bug: 16810766 Change-Id: Ib14c70bd0550420cecfa86dea501d13a1a91e296
* Move getPrevWordsInfoFromNthPreviousWord to PrevWordsInfoUtils.Keisuke Kuroyanagi2014-07-071-76/+3
| | | | | Bug: 14425059 Change-Id: Id37022ac6c1545d6845abfbcdb7ed47f0e250eec
* [SD7] Actually check for script.Jean Chalard2014-07-011-3/+7
| | | | | | | ...also implement the check for Hebrew and Arabic. Bug: 15840116 Change-Id: Ia6433d7d98038ade64c171be4fe4b3f094111fac
* [SD6] Inline a constant and remove logic become uselessJean Chalard2014-07-011-21/+8
| | | | | Bug: 15840116 Change-Id: I545cc9083aa4e2fd7cbbd1fbc02e1e382482db7c
* [SD5(2)] Remove useless argsJean Chalard2014-07-011-16/+2
| | | | | Bug: 15840116 Change-Id: I1123426fbd9d420c1be64ccc917a5f870e70e6fa
* Revert "[SD5] Remove useless args"Ken Wakasa2014-07-011-2/+16
| | | | | | | This reverts commit 1d300239612591879d535c20ade1f2712048170e that broke the build. Bug: 15840116 Change-Id: I0a5fa7dea2b418d19df24b2b31ed96bf192d45c0
* [SD5] Remove useless argsJean Chalard2014-06-301-16/+2
| | | | | Bug: 15840116 Change-Id: Ib3380cfc9d343c6f8953bba03af3801142bc3bdb
* Merge "Revert "Move getPrevWordsInfoFromNthPreviousWord to StringUtlis.""Ken Wakasa2014-06-261-1/+76
|\
| * Revert "Move getPrevWordsInfoFromNthPreviousWord to StringUtlis."Ken Wakasa2014-06-271-1/+76
| | | | | | | | | | | | | | This reverts commit ba463c9a66f75e8d00f4658e32b763eb54215231 that broke the dicttool build. Bug: 14425059 Change-Id: Ie1685587104d26e4416624747c97f6087c13388a
* | Merge "Move getPrevWordsInfoFromNthPreviousWord to StringUtlis."Keisuke Kuroyanagi2014-06-261-76/+1
|\|
| * Move getPrevWordsInfoFromNthPreviousWord to StringUtlis.Keisuke Kuroyanagi2014-06-271-76/+1
| | | | | | | | | | Bug: 14425059 Change-Id: I3eb24e840c165e43f68c2a60fccf9974affb57a6
* | Merge "Find multiple previous word information to support n-gram."Keisuke Kuroyanagi2014-06-261-46/+59
|\|
| * Find multiple previous word information to support n-gram.Keisuke Kuroyanagi2014-06-271-46/+59
| | | | | | | | | | | | Bug: 14425059 Change-Id: Ieace636334a9b2a094527341d4fcfc05958296c5
* | Add a relevant suggestion to period-checkable requestsJean Chalard2014-06-271-3/+2
|/ | | | | Bug: 15412461 Change-Id: Ibf37df4d31141a7e43b54d6342e7861eedb1c03b
* Make PrevWordsInfo have multiple words' information.Keisuke Kuroyanagi2014-06-251-1/+1
| | | | | Bug: 14425059 Change-Id: I2bd6a872904a44b80f638a13d91a97559217cc1a
* Fix a bug on rotation with selection.Jean Chalard2014-06-061-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The symptom : when text is selected and the device is rotated, sometimes the keyboard sets the word as being composed around the start of the selection. Upon the next rotation this ends up with the keyboard committing some text in place of the selection. The cause : another bug in the framework with rotation >.> The keyboard receives a call to startInput with a wrong cursor position, namely one that does not represent a selection. The keyboard sets a composition according to this wrong data. When the keyboard is rotated again, it commits the text, which takes the place of the selection. The solution : actually when restarting input the keyboard realizes that the cursor position is wrong. We cancel composition at that time. For robustness, this change also implements two other defensive changes : upon call to onUpdateSelection, we actually realize that the previous values were wrong, so we also fix it at that time, and in addition, when rotating, we finishComposingText() instead of commitText() which is less dangerous. Implementing this later change also allows us to let less internal variables from InputLogic escape to LatinIME, so it's also a good change for design. Bug: 14140799 Change-Id: Ib10de18e53e376ac1bbc8487e13d969828483346
* Make prevWord = null if the next word starts from a connector.Keisuke Kuroyanagi2014-05-301-1/+17
| | | | | | | | | | This fixes PunctuationTests# testAutoCorrectionWithSingleQuotesAround. Bug: 14119293 Bug: 15334309 Change-Id: I604c21a21e89a5fc431fd56ab7b6ad03f4736b01
* Remove researcher loggerTadashi G. Takaoka2014-05-291-32/+0
| | | | | | | This CL must be checked in together with I5cc76807e3. Bug: 15318007 Change-Id: I61423c3377ddc299fb332e742d6626c2e47145bb
* Introduce EMPTY_PREV_WORDS_INFO and BEGINNING_OF_SENTENCE.Keisuke Kuroyanagi2014-05-231-6/+6
| | | | | Bug: 14119293 Change-Id: I5020e5f0aa64bc3e97b3a3c2c07a60c8b765ed64
* Add beginning of sentence information in PrevWordsInfo.Keisuke Kuroyanagi2014-05-211-21/+34
| | | | | | Bug: 14119293 Bug: 14425059 Change-Id: I65320920e840082b0b697bb621676716d0933e0c
* Rework the logic that tells if the cursor touches wordsJean Chalard2014-03-201-7/+17
| | | | | Bug: 13312942 Change-Id: I6be6a558bbc6c88508150f9c25cadbd0240ff88e
* Don't reset the value needed before it's usedJean Chalard2014-03-101-2/+2
| | | | | | | Nice recipe for failure Bug: 13387534 Change-Id: Ida1978449c1997587b2ec0955c5c94fcef336121