| Commit message (Expand) | Author | Age | Files | Lines |
* | Remove mPrevWordsInfo•••This member has outlived its usefulness. It's not clear now that it
makes things really faster, but it does bring a lot of complexity
that we can avoid by removing it.
Change-Id: Ifbc8094a45b56b958fe165b1930f4cc358a97721
|  Jean Chalard | 2014-06-10 | 1 | -44/+10 |
* | When starting input, include the literal word in suggestions•••This change also includes a fix that has suggestions re-computed
when the typed word is included but no prior suggestions were
found in spans.
Bug: 2349475
Change-Id: Ic06e6ac492507126ffc1e96a5f396c971b567272
|  Jean Chalard | 2014-06-06 | 1 | -10/+13 |
* | Merge "Fix a bug on rotation with selection." |  Jean Chalard | 2014-06-06 | 1 | -3/+13 |
|\ |
|
| * | Fix a bug on rotation with selection.•••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
|  Jean Chalard | 2014-06-06 | 1 | -3/+13 |
* | | Rename some boolean variables for readability•••Change-Id: I434cf507e68349b9893ac084645986d6b3d7da91
|  Tadashi G. Takaoka | 2014-06-05 | 1 | -2/+2 |
|/ |
|
* | Don't recapitalize just at the start of input•••Bug: 13283555
Change-Id: Iab0466425dc44779d110f2cd29d725d22eb04827
|  Jean Chalard | 2014-06-04 | 1 | -7/+5 |
* | Refactoring•••Rename `initialize' to `start' and `deactivate' to `stop' for ease
of understanding. It's completely bizarre that you can "deactivate"
a recapitalization and that will just restart a new one the next time
you press shift. Start and stop are easier to understand.
Change-Id: I455175aebfcfe11198c6c61bf2c6b19efc3f82df
|  Jean Chalard | 2014-06-04 | 1 | -7/+7 |
* | Make prevWord = null if the next word starts from a connector.•••This fixes PunctuationTests#
testAutoCorrectionWithSingleQuotesAround.
Bug: 14119293
Bug: 15334309
Change-Id: I604c21a21e89a5fc431fd56ab7b6ad03f4736b01
|  Keisuke Kuroyanagi | 2014-05-30 | 1 | -8/+10 |
* | Limit recapitalization for reasonable performance.•••At 100k text, it's reasonably fast (less than 1s on latest hardware).
Bug: 12913404
Change-Id: I426b918b2610af24364934a1c37a7314f1142ad0
|  Jean Chalard | 2014-05-30 | 1 | -9/+12 |
* | Merge "Remove LatinImeLogger and UsabilityStudyLogUtils" |  Tadashi G. Takaoka | 2014-05-29 | 1 | -40/+0 |
|\ |
|
| * | Remove LatinImeLogger and UsabilityStudyLogUtils•••This CL must be checked in together with Ie52007dc5c.
Bug: 15318007
Change-Id: Ie665f296e43b5485ae8ba7e949aa262794ee0dcc
|  Tadashi G. Takaoka | 2014-05-29 | 1 | -40/+0 |
* | | Fix: too many calls to getSuggestedWordsForTypingInput•••This reverts commit d941ea18 and builds on it to fix the behavior
of the broken cases.
It also fixes a small, related bug that probably has existed for
a very long time: predictions not displayed when cancelling
double-space-to-period.
Bug: 15148015
Change-Id: I1f9358f8b6f5804f831643611576be347e83999d
|  Jean Chalard | 2014-05-29 | 1 | -2/+11 |
|/ |
|
* | Remove researcher logger•••This CL must be checked in together with I5cc76807e3.
Bug: 15318007
Change-Id: I61423c3377ddc299fb332e742d6626c2e47145bb
|  Tadashi G. Takaoka | 2014-05-29 | 1 | -75/+1 |
* | Stop directly checking SuggestedWordInfo.mKind•••Bug: 15298959
Change-Id: Iba6eeb7981634e7ddb896404978b30f6efd6e72f
|  Keisuke Kuroyanagi | 2014-05-28 | 1 | -3/+3 |
* | Merge "Use whether it's exact match to detect distracters." |  Keisuke Kuroyanagi | 2014-05-26 | 1 | -1/+1 |
|\ |
|
| * | Use whether it's exact match to detect distracters.•••Bug: 13142176
Change-Id: Id5b7286d28897931f7bfe571be45d46ffeef4adf
|  Keisuke Kuroyanagi | 2014-05-26 | 1 | -1/+1 |
* | | Keep showing important notice when "show correction suggestions" is off•••When
1. The important notice message is shown on the suggestions strip.
2. And the "Show correction suggestions" settings is off.
we will keep showing the important notice message on the suggestion
strip.
Bug: 13741460
Change-Id: I411007ab1e5e6959b6cdba7a6601a84635259313
|  Tadashi G. Takaoka | 2014-05-26 | 1 | -1/+2 |
|/ |
|
* | Use Java 7 diamond operator•••Change-Id: If16ef50ae73147594615d0f49d6a22621eaf1aef
|  Tadashi G. Takaoka | 2014-05-24 | 1 | -5/+4 |
* | Merge "Introduce EMPTY_PREV_WORDS_INFO and BEGINNING_OF_SENTENCE." |  Keisuke Kuroyanagi | 2014-05-23 | 1 | -2/+3 |
|\ |
|
| * | Introduce EMPTY_PREV_WORDS_INFO and BEGINNING_OF_SENTENCE.•••Bug: 14119293
Change-Id: I5020e5f0aa64bc3e97b3a3c2c07a60c8b765ed64
|  Keisuke Kuroyanagi | 2014-05-23 | 1 | -2/+3 |
* | | Merge "Revert "Fix: too many calls to getSuggestedWordsForTypingInput"" |  Ken Wakasa | 2014-05-23 | 1 | -6/+3 |
|\ \
| |/
|/| |
|
| * | Revert "Fix: too many calls to getSuggestedWordsForTypingInput"•••This reverts commit 7aa895e2e3edb59620c28354c7d7746b93964b10.
Bug: 15148015
Change-Id: If39c4f8b897f247d4eb935923d55d81def4c8fb4
|  Ken Wakasa | 2014-05-23 | 1 | -6/+3 |
* | | Have DictionaryFacilitator in LatinIME.•••Bug: 13142176
Bug: 15094186
Change-Id: I9da67352ff2abcd289fa5885389895ced878a413
|  Keisuke Kuroyanagi | 2014-05-23 | 1 | -11/+14 |
* | | Merge "Fix: too many calls to getSuggestedWordsForTypingInput" |  Jean Chalard | 2014-05-22 | 1 | -3/+6 |
|\| |
|
| * | Fix: too many calls to getSuggestedWordsForTypingInput•••Bug: 15148015
Change-Id: Idcb729734bd62b537fb81acf8bdb5cae4470404c
|  Jean Chalard | 2014-05-22 | 1 | -3/+6 |
* | | Add beginning of sentence information in PrevWordsInfo.•••Bug: 14119293
Bug: 14425059
Change-Id: I65320920e840082b0b697bb621676716d0933e0c
|  Keisuke Kuroyanagi | 2014-05-21 | 1 | -16/+19 |
* | | Use PrevWordsInfo for get/add/remove n-gram(bigram) entry.•••Bug: 14119293
Bug: 14425059
Bug: 15102610
Change-Id: Ib482390f8d15b37d44a1a54e8c243a9eaec3f815
|  Keisuke Kuroyanagi | 2014-05-21 | 1 | -17/+21 |
|/ |
|
* | Revert "Use PrevWordsInfo for get/add/remove n-gram(bigram) entry."•••This reverts commit ff50b39176370ab80a33bfdcf9979603c08a88b3.
Bug: 14119293
Bug: 14425059
Bug: 15102610
Change-Id: If278b4ab236e38d20d8cdc0761b0438911bd4ff9
|  Ken Wakasa | 2014-05-20 | 1 | -19/+16 |
* | Use PrevWordsInfo for get/add/remove n-gram(bigram) entry.•••Bug: 14119293
Bug: 14425059
Change-Id: I12e9ba977c153b514c6591ab52940712fd0874e3
|  Keisuke Kuroyanagi | 2014-05-20 | 1 | -16/+19 |
* | Fix a bug where changing languages kills the composition.•••Bug: 14926896
Change-Id: Id2a06c68a51f833db4d8bbc858a232c2696afb6e
|  Jean Chalard | 2014-05-14 | 1 | -2/+2 |
* | Don't send empty composing texts.•••Bug: 13160467
Change-Id: I5e3204ac8aaa18fe9219cc6b564b832ab6cf94fc
|  Jean Chalard | 2014-05-14 | 1 | -1/+5 |
* | Merge "Add freq=0 words to user history depending on the option." |  Keisuke Kuroyanagi | 2014-05-12 | 1 | -1/+1 |
|\ |
|
| * | Add freq=0 words to user history depending on the option.•••Bug: 14686585
Change-Id: Ia1c3adb1bcc625788016e378e1ac7b0792ea9f4e
|  Keisuke Kuroyanagi | 2014-05-12 | 1 | -1/+1 |
* | | Merge "Improve multi-word suggestion handling for user history." |  Keisuke Kuroyanagi | 2014-05-12 | 1 | -3/+4 |
|\| |
|
| * | Improve multi-word suggestion handling for user history.•••Bug: 12902664
Change-Id: I03deed16047f0beaba8932d87c6129ec8061ba9c
|  Keisuke Kuroyanagi | 2014-05-12 | 1 | -3/+4 |
* | | Create the path for specifying combining rules in the subtype•••Also add the skeleton of the combining rules for Myanmar
reordering as a pilot.
Bug: 13945569
Change-Id: Iabeb6838f45f1efff482143595ce5b48543e0c43
|  Jean Chalard | 2014-05-12 | 1 | -1/+17 |
|/ |
|
* | Introduce clustering punctuation•••Bug: 10081730
Change-Id: If198fa3df68ecb267da0a278da41fd509d6165f1
|  Jean Chalard | 2014-05-01 | 1 | -10/+13 |
* | Fix handling shift + enter•••Bug: 14278758
Change-Id: Id77c99b83cc69855972b27c9cebd482c8cbd3539
|  Tadashi G. Takaoka | 2014-04-23 | 1 | -1/+3 |
* | Restart suggestions when restarting input•••Bug: 6851364
Change-Id: I3052924053f37f56c9c3953051aa1b22224a2b2e
|  Jean Chalard | 2014-04-16 | 1 | -0/+4 |
* | Fix updating the shift state upon backspace•••Bug: 13514349
Change-Id: If4c9db12b0ab5be676f7a2f72715f469066ee537
|  Jean Chalard | 2014-04-14 | 1 | -11/+11 |
* | Merge "[IL133] Avoid calling a dangerous method from outside." |  Jean Chalard | 2014-04-10 | 1 | -3/+6 |
|\ |
|
| * | [IL133] Avoid calling a dangerous method from outside.•••We want to remove all calls to this as it lets internal values
escape, but there is some refactoring to do to finish this.
Bug: 8636060
Change-Id: Iedba6afe4719bc0add868714a1ee5a04b7ead33e
|  Jean Chalard | 2014-04-03 | 1 | -3/+6 |
* | | Fix some flaky tests.•••In tests, we create many instances of LatinIME, but we never
destroy them. That means we never close the dictionaries nor
the handlers.
This change calls onDestroy, which closes all dictionaries, and
adds some code to finish the handlers.
Change-Id: I942517a2a940c54256b08763f6b38f5b55809f55
|  Jean Chalard | 2014-04-10 | 1 | -0/+11 |
* | | [CB23] Merge add and deleteLast•••Bug: 13406701
Change-Id: Id9a2e4144b1908ad27d78228ee19e6fc4d4d669f
|  Jean Chalard | 2014-04-07 | 1 | -2/+2 |
* | | Merge "[IL132] Remove some calls that let a value escape." |  Jean Chalard | 2014-04-04 | 1 | -3/+3 |
|\| |
|
| * | [IL132] Remove some calls that let a value escape.•••Bug: 8636060
Change-Id: I696514934586ee71e734fd974026af8b2c866127
|  Jean Chalard | 2014-04-03 | 1 | -3/+3 |
* | | Fix a bug with double-space-to-period•••Bug: 13778001
Change-Id: I3ebd57950cdfacbbcdc64ed214c0590519a0665c
|  Jean Chalard | 2014-04-03 | 1 | -7/+5 |
|/ |
|
* | [IL131] Cleanup some cruft•••Bug: 8636060
Change-Id: I169618b50532df68e973e14b3f114c6c19714c84
|  Jean Chalard | 2014-04-03 | 1 | -9/+1 |
* | [IL130] Remove a useless local•••Bug: 8636060
Change-Id: I36ba3e00630bef644fc4671d0714490b5982f6a1
|  Jean Chalard | 2014-04-02 | 1 | -3/+1 |
* | [IL129] Change some confusing method names.•••Bug: 8636060
Change-Id: I2f55fc23d7e05b63582a3f0660b58635eae5ec59
|  Jean Chalard | 2014-04-02 | 1 | -2/+2 |