aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/latin (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Add preference dependency to gesture input related optionsTadashi G. Takaoka2013-04-151-4/+0
| | | | | Bug: 8613600 Change-Id: I279d3264080aede59203205c45763c7f8ff08ae6
* Merge "Reset vertical correction when more suggestions are in modal mode"Tadashi G. Takaoka2013-04-152-2/+8
|\
| * Reset vertical correction when more suggestions are in modal modeTadashi G. Takaoka2013-04-152-2/+8
| | | | | | | | | | Bug: 8605150 Change-Id: Iadbebfb80e765c97e58e8524eb87419f2a9e41d4
* | Fix Binary dict testsJean Chalard2013-04-151-4/+4
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are two problems here. The first one is the tests would send an invalid unicode character. Although we could want dicttool to handle this more gracefully, it's fine for now. The second problem is much more serious. If a node has more than 128 children, then the java code will crash trying to read the dictionary back because of a bug that this change fixes. In theory, it's possible that happens when we try to load the user history dictionary back from the disk - native code is not affected so there is no other point that may cause a problem. In the practice, that means you'd need to have 129 words with a common prefix (including empty string) but all different after this. It's almost impossible with Google Keyboard since there are only so many keys on the keyboard that you can make a word out of, and then again you'd have to do it repeatedly until it actually enters the user history dictionary, wait for it to get saved on the disk. The bad news is, if you manage to get this far, the keyboard will crash every time and won't be able to get up until you clear data for the package. The good news is, the dictionary itself is not corrupted and only the reading code is wrong. So updating to a newer version would actually even recover from this situation. All in all, considering how almost-impossible this is to trigger, I don't think even a single user actually did hit this bug. Bug: 8583091 Change-Id: Iabb2a7f47cbd9ed3193d2a3487318d280753e071
* Merge "Remove the dialog to insert words to the dictionary."Jean Chalard2013-04-122-22/+12
|\
| * Remove the dialog to insert words to the dictionary.Jean Chalard2013-04-122-22/+12
| | | | | | | | | | Bug: 8562529 Change-Id: Ibd853f0cb995e7fb35f8730eafcef73ead58c869
* | Merge "Restart suggestions when the cursor moves."Jean Chalard2013-04-126-14/+102
|\ \
| * | Restart suggestions when the cursor moves.Jean Chalard2013-04-126-14/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-123-2/+48
|/ / | | | | | | | | Bug: 8561056 Change-Id: I91fb0ac76dabfc841bafc0e2c0ea9f63e6f604f9
* | Merge "Make shortcut-only user dict entry non-words."Jean Chalard2013-04-123-6/+8
|\ \
| * | Make shortcut-only user dict entry non-words.Jean Chalard2013-04-123-6/+8
| |/ | | | | | | | | Bug: 7369456 Change-Id: Iecd6f6d617f31bd2aa6749516711aeb1f0549622
* | Merge "Add a class to encapsulate getting the metadata URI"Jean Chalard2013-04-122-1/+29
|\ \ | |/ |/|
| * Add a class to encapsulate getting the metadata URIJean Chalard2013-04-112-1/+29
| | | | | | | | | | Bug: 8376173 Change-Id: Icaac8ad412e0180c44e6a955943e3b208981b2ea
* | Merge "Add some utility functions."Jean Chalard2013-04-111-0/+71
|\ \
| * | Add some utility functions.Jean Chalard2013-04-111-0/+71
| |/ | | | | | | | | Bug: 7657025 Change-Id: I4889721b5348c77ed56c5157557e9988dea48a02
* | Merge "Add TypefaceUtils"Tadashi G. Takaoka2013-04-113-12/+10
|\ \
| * | Add TypefaceUtilsTadashi G. Takaoka2013-04-113-12/+10
| |/ | | | | | | Change-Id: I2d939bc1e3279253902b28ff12bbf7861716b64d
* | Merge "Use SparseIntArray instead of TreeMap<Integer, Integer>"Tadashi G. Takaoka2013-04-111-42/+24
|\ \
| * | Use SparseIntArray instead of TreeMap<Integer, Integer>Tadashi G. Takaoka2013-04-101-42/+24
| | | | | | | | | | | | Change-Id: I56a64228a5fd85d440d0d08ac67cba8bf2e5690d
* | | Merge "Fix two bugs in dicttool"Jean Chalard2013-04-111-3/+6
|\ \ \ | |_|/ |/| |
| * | Fix two bugs in dicttoolJean Chalard2013-04-101-3/+6
| | | | | | | | | | | | | | | | | | | | | Both bugs only affect debug mode. One has the wrong object tested with equals, the other has the iteration failing in some cases. Change-Id: Ie9100d257a3f9e3be340cf3e38116f63417bdc1a
* | | Merge "Fix a bug where a space would not be inserted before a gesture"Jean Chalard2013-04-101-1/+2
|\ \ \
| * | | Fix a bug where a space would not be inserted before a gestureJean Chalard2013-04-101-1/+2
| |/ / | | | | | | | | | | | | Bug: 8583675 Change-Id: I14924fef5663f48481994c02c56be45f9f85be65
* / / Use private lock object instead of synchronized methodTadashi G. Takaoka2013-04-101-22/+31
|/ / | | | | | | Change-Id: Ifc62bacbd0583a7d102009681a94bdd9ccff7d47
* | Fix a bug where autocorrection status would be wrong.Jean Chalard2013-04-106-9/+10
| | | | | | | | Change-Id: Ic220129dc59f585164dbf63591cd1c96de17fe6f
* | Don't override EXTRA_SHOW_FRAGMENT if existsTadashi G. Takaoka2013-04-101-1/+4
| | | | | | | | | | Bug: 8541190 Change-Id: I37d3de12558efd55f157389d2e0b51fc83dc5da1
* | Merge "Separate StringUtils.capitalizeFirstCharacter"Tadashi G. Takaoka2013-04-102-4/+12
|\ \
| * | Separate StringUtils.capitalizeFirstCharacterTadashi G. Takaoka2013-04-102-4/+12
| |/ | | | | | | | | Bug: 8582061 Change-Id: Iac8f65defdd92d7df533bdf0e2937ad897d96363
* / Hide Setup Wizard icon for stock keyboardTadashi G. Takaoka2013-04-102-1/+12
|/ | | | | Bug: 8581647 Change-Id: Id530f1a33a5c919f4a732ec0d545dfbe76e56316
* Merge "Refactor SpellCheckerProximityInfo as extending ProximityInfo"Tadashi G. Takaoka2013-04-102-39/+44
|\
| * Refactor SpellCheckerProximityInfo as extending ProximityInfoTadashi G. Takaoka2013-04-022-39/+44
| | | | | | | | Change-Id: I544eec9d661796d57749dbaa2163bc1e2a891353
* | Merge "Offer to add any OOV typed word to the dictionary."Jean Chalard2013-04-092-7/+11
|\ \
| * | Offer to add any OOV typed word to the dictionary.Jean Chalard2013-04-092-7/+11
| | | | | | | | | | | | | | | | | | | | | That includes gestures, which used not to work. Bug: 8532637 Change-Id: I04606565d7000faadf954c4a806c39d4d162a2c1
* | | Merge "Fix bugs and add tests"Jean Chalard2013-04-091-9/+20
|\ \ \
| * | | Fix bugs and add testsJean Chalard2013-04-051-9/+20
| | | | | | | | | | | | | | | | Change-Id: I6b56b91ace57f4a49584b5dceb71b145859f839e
* | | | Merge "Do not record log split time when handling separator"Kurt Partridge2013-04-091-1/+0
|\ \ \ \ | |_|/ / |/| | |
| * | | Do not record log split time when handling separatorKurt Partridge2013-04-011-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously handleSeparator() caused the ResearchLogger to mark the time at which a LogUnit should be broken. However, this causes the motion data associated with a separator to be associated with the LogUnit of the previous word. This change corrects this bug. Change-Id: I8b4d4fa6de2a013de9e2a28bb668c446a07f1957
* | | | Update commentsKen Wakasa2013-04-081-1/+1
| |/ / |/| | | | | | | | Change-Id: I6aef328ec9041666cc168c8323bed04d39591b71
* | | Move a generic string utility to StringUtilsJean Chalard2013-04-053-36/+36
| | | | | | | | | | | | Change-Id: I9cc2e9a7ac0b3346af40bcb083f939333336cf09
* | | Revert "[DO NOT MERGE] Activate hardware keyboard handling"Jean Chalard2013-04-021-1/+1
| | | | | | | | | | | | | | | | | | | | | This reverts commit c741f1f83b98293bf0040b6b81e95b75ecf3f6bc DO NOT MERGE is ignored and this is merged anyway :( Change-Id: Ie044cdfc6021933379ec97ad7346c5c8591f70bf
* | | [DO NOT MERGE] Activate hardware keyboard handlingJean Chalard2013-04-021-1/+1
| | | | | | | | | | | | Change-Id: Ifdec9338a454291621b2b87c899f5d4301480b27
* | | Merge "Reorder ResearchLogger call"Kurt Partridge2013-04-011-2/+2
|\ \ \
| * | | Reorder ResearchLogger callKurt Partridge2013-03-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, ResearchLogger#onWordFinished() was called with an outdated parameter value for isBatchMode, causing it to report false even for gestures. This changes fixes this problem. Change-Id: Ifcabee236ba5fe20376ad882155d3f3142cd7613
* | | | am b8869ec1: am 0fcc4960: Deactivate hardware keyboard handling.Jean Chalard2013-04-011-1/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | * commit 'b8869ec1e39c1751825841e790d21fc854dca9ce': Deactivate hardware keyboard handling.
| * | | Deactivate hardware keyboard handling.Jean Chalard2013-04-011-1/+1
| |/ / | | | | | | | | | | | | | | | Bug: 8000119 Change-Id: I021481343d6d87efd8ff6110e9aa0906a1a3d11d
* | | Merge "Ask the client to make itself known when it's not"Jean Chalard2013-04-012-1/+74
|\ \ \
| * | | Ask the client to make itself known when it's notJean Chalard2013-04-012-1/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upon invoking the settings of the dictionary pack with an unknown client, we now launch an intent to ask the client to make itself known. This change also includes the code that receives this intent and acts upon it. Bug: 8492879 Change-Id: I2c6496dea845646961ecafcf64e282cb93ee91dc
* | | | Merge "Fix two nasty bugs with surrogate pairs."Jean Chalard2013-04-011-13/+13
|\ \ \ \ | |/ / / |/| | |
| * | | Fix two nasty bugs with surrogate pairs.Jean Chalard2013-04-011-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The important bug is in findWordInTree. The problem, which is not obvious, is that we were calling codePointAt() with the code point index in the string, instead of the char index. The other bug this change fixes was harmless in the practice, because it's in the iteration which is only used for debug and pretty printing purposes. It's very similar in that it would substract a length in code point to a length in chars and truncate a StringBuilder at that length, so it would fail in a quite similar manner. This changes the meaning of the "length" attribute in Position, but it's clearer this way anyway. Bug: 8450145 Change-Id: If396f883a9e6449de39351553ba83f5be5bd30f0
* | | | Fix an NPEJean Chalard2013-03-291-18/+10
|/ / / | | | | | | | | | | | | | | | | | | | | | Also remove an unused method and make 2 private variables final. Bug: 8461093 Change-Id: I676d93fda28e82eb5a6978f2e7ad5a0add2cbecf
generated by cgit v1.2.3-83-g751a (git 2.47.1) at 2025-08-02 21:01:39 +0000