aboutsummaryrefslogtreecommitdiffstats
path: root/java/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Don't do recorrection when suggestions are off.Jean Chalard2013-05-011-2/+7
| | | | | Bug: 8767981 Change-Id: If5c775e488b91ae03ecce2673c99918ac8134928
* Merge "Fix an NPE in recorrection" into jb-mr2-devJean Chalard2013-04-191-8/+11
|\
| * Fix an NPE in recorrectionJean Chalard2013-04-191-8/+11
| | | | | | | | | | | | | | | | | | Also, theoretically, we don't want to retrieve older suggestions if there are no results for a recorrection - that would look random. This bug addresses this too. Bug: 8657919 Change-Id: I44f36d34a708a968ab71fa0592da57f4c97d4b7f
* | Merge "Fix an NPE in recapitalize" into jb-mr2-devJean Chalard2013-04-191-2/+5
|\ \
| * | Fix an NPE in recapitalizeJean Chalard2013-04-191-2/+5
| |/ | | | | | | | | Bug: 8657736 Change-Id: I459d1c200826c1c394f8207475ecf60a4f356793
* | Merge "Fix an infinite loop" into jb-mr2-devKen Wakasa2013-04-181-1/+0
|\ \
| * | Fix an infinite loopJean Chalard2013-04-181-1/+0
| |/ | | | | | | | | | | | | | | | | What do you mean "Can't happen"? It happens all the time - the empty string is the default ID, and it needs to be updated like everyone else. Bug: 8651858 Change-Id: I5a2f2ebb5b2ef08b27f26be8fb2c3d2f231ebcfc
* / Initialize dictionaries at onStartInputViewJean Chalard2013-04-182-5/+11
|/ | | | | Bug: 8344402 Change-Id: I41b8440f0d72a66fde3f6feb0db8239c9bb10ea3
* Fix bug to update last interpolated index of GestureStrokeTadashi G. Takaoka2013-04-171-5/+5
| | | | | Bug: 8556775 Change-Id: Ie12e0dc9bfeae45f6883580801e07d44be1d1d33
* Merge "Don't insert automatic spaces when text looks like a URL"Jean Chalard2013-04-167-7/+86
|\
| * Don't insert automatic spaces when text looks like a URLJean Chalard2013-04-167-7/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Merge "Add shadow to gesture preview trail"Tadashi G. Takaoka2013-04-162-8/+39
|\ \ | |/ |/|
| * Add shadow to gesture preview trailTadashi G. Takaoka2013-04-162-8/+39
| | | | | | | | | | Bug: 7167303 Change-Id: I8f76fe0aec6cb32388e7c85d04516d1dd7d28c29
* | Merge "Small cleanup"Jean Chalard2013-04-161-3/+3
|\ \
| * | Small cleanupJean Chalard2013-04-161-3/+3
| | | | | | | | | | | | Change-Id: Iafdbe305712c8a571f0f5abc2032df086d7ac015
* | | Merge "Don't recapitalize on shift while on symbol layout"Jean Chalard2013-04-161-1/+6
|\ \ \ | |/ / |/| |
| * | Don't recapitalize on shift while on symbol layoutJean Chalard2013-04-161-1/+6
| | | | | | | | | | | | Change-Id: Ibf2cdedf510c296561bb5eeb6ff18c3414d6d6f4
* | | Match the keyboard state to the recapitalize state.Jean Chalard2013-04-164-8/+59
|/ / | | | | | | | | Bug: 7657025 Change-Id: I2f8fe7fc4596a498322ba5ccabbd0c18a2bc36cf
* | Merge "Clean up RecapitalizeStatus"Jean Chalard2013-04-162-29/+44
|\ \
| * | Clean up RecapitalizeStatusJean Chalard2013-04-162-29/+44
| | | | | | | | | | | | Change-Id: Ib4d002f90cd3a0e9ad4c04b883b0c1f05ada3ccf
* | | Merge "Fix an NPE"Jean Chalard2013-04-161-0/+1
|\ \ \ | |/ / |/| |
| * | Fix an NPEJean Chalard2013-04-161-0/+1
| | | | | | | | | | | | | | | Bug: 8620678 Change-Id: Ia91761f33518a010b2844d34523cadece261c67e
* | | Merge "Fix an NPE"Jean Chalard2013-04-161-2/+2
|\ \ \ | |_|/ |/| |
| * | Fix an NPEJean Chalard2013-04-161-2/+2
| |/ | | | | | | | | Bug: 8620758 Change-Id: I39f07939ebb6148a48aae5b915ee807002a5b02d
* | Merge "Small performance improvement / minor bugfix"Jean Chalard2013-04-151-3/+3
|\ \
| * | Small performance improvement / minor bugfixJean Chalard2013-04-151-3/+3
| |/ | | | | | | | | | | | | | | | | The only place where it's used is checked for nullity. Also, it's possible, also difficult, to match a different recapitalize with the old code, triggering a bug that this fixes. Change-Id: I717d6df489025c75d1caca290a9086c3b39a9306
* | Merge "Remove a useless function"Jean Chalard2013-04-151-11/+0
|\ \
| * | Remove a useless functionJean Chalard2013-04-151-11/+0
| |/ | | | | | | | | Bug: 8583091 Change-Id: I9195d68e44e9a282e25ccd2978d7b4088f600170
* / Remove gesture suggestions with an INT_MIN scoreJean Chalard2013-04-151-0/+12
|/ | | | | Bug: 7980115 Change-Id: I8de31068992aec712eaad132857d9dc61f28e30a
* Merge "Have Latin IME re-capitalize a selected string"Jean Chalard2013-04-154-1/+217
|\
| * Have Latin IME re-capitalize a selected stringJean Chalard2013-04-154-1/+217
| | | | | | | | | | | | | | | | | | | | 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
* | If there are no suggestion span, recompute suggestions.Jean Chalard2013-04-153-16/+88
|/ | | | | Bug: 8084810 Change-Id: I1743c09c43ca6835bb2f607684b037bf17d36335
* Merge "Reject a previously user-refused suggestion."Jean Chalard2013-04-153-0/+27
|\
| * Reject a previously user-refused suggestion.Jean Chalard2013-04-153-0/+27
| | | | | | | | | | | | | | | | | | If the user gestures a word, then hits backspace in disapproval, and gestures about the same thing again, make sure that we don't suggest the same thing again. Bug: 7549311 Change-Id: I793bc4df7c3841fa8f2f4146707c26e873f374c1
* | Remove unnecessary harmful finish() from SettingsActivityTadashi G. Takaoka2013-04-151-7/+2
| | | | | | | | | | Bug: 8560722 Change-Id: I825b135bd13aca17e712330660978b834648dbc3
* | Merge "Ignore modifier key while sliding key input"Tadashi G. Takaoka2013-04-151-3/+3
|\ \
| * | Ignore modifier key while sliding key inputTadashi G. Takaoka2013-04-151-3/+3
| | | | | | | | | | | | | | | Bug: 8284624 Change-Id: I839d7a769d98f29be647dbe511ba3f73ca44446d
* | | Merge "Suppress calling onPressKey and onReleaseKey while detecting gesture"Tadashi G. Takaoka2013-04-151-2/+2
|\ \ \
| * | | Suppress calling onPressKey and onReleaseKey while detecting gestureTadashi G. Takaoka2013-04-151-2/+2
| | | | | | | | | | | | | | | | | | | | Bug: 8284624 Change-Id: I14470bb2373965aefa921b2c94948450bfa438e3
* | | | 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-153-3/+9
|\ \ \
| * | | Reset vertical correction when more suggestions are in modal modeTadashi G. Takaoka2013-04-153-3/+9
| |/ / | | | | | | | | | | | | Bug: 8605150 Change-Id: Iadbebfb80e765c97e58e8524eb87419f2a9e41d4
* | | Merge "Fix Binary dict tests"Jean Chalard2013-04-151-4/+4
|\ \ \ | |/ / |/| |
| * | 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 "Avoid NPE"Kurt Partridge2013-04-121-3/+3
|\ \ | |/ |/|
| * Avoid NPEKurt Partridge2013-03-261-3/+3
| | | | | | | | | | | | Initialize FeedbackLog earlier to avoid NPE Change-Id: I646944c68a7d092345def16060b2a4edd8c7dbef
* | Merge "Remove the dialog to insert words to the dictionary."Jean Chalard2013-04-123-22/+63
|\ \
| * | Remove the dialog to insert words to the dictionary.Jean Chalard2013-04-123-22/+63
| | | | | | | | | | | | | | | Bug: 8562529 Change-Id: Ibd853f0cb995e7fb35f8730eafcef73ead58c869
* | | Merge "Restart suggestions when the cursor moves."Jean Chalard2013-04-127-15/+103
|\ \ \
| * | | Restart suggestions when the cursor moves.Jean Chalard2013-04-127-15/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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