aboutsummaryrefslogtreecommitdiffstats
path: root/java/src (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
* | | 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
* | | | 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 "Better isolate ResearchLogging data"Kurt Partridge2013-04-111-0/+3
|\ \ \
| * | | Better isolate ResearchLogging dataKurt Partridge2013-04-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Calls to LatinIME#onStartInputViewInternal log important information about the context in which an IME is used. This is reported as a single LogStatement. Previously, this was not placed into a separate LogUnit, and was mixed in with general word data. This change wraps this LogStatement in its own LogUnit. Change-Id: I0fecd41c8a1de622a764cc4b5d6902336697046c
* | | | 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
* | | | Add debug message to detect unusual zero width conditionTadashi G. Takaoka2013-04-111-0/+12
| | | | | | | | | | | | | | | | | | | | Bug: 8452013 Change-Id: I0169dc6b0d08d1b4d9e01a5fa2ad01e9db292bda
* | | | Merge "Add TypefaceUtils"Tadashi G. Takaoka2013-04-118-128/+178
|\ \ \ \
| * | | | Add TypefaceUtilsTadashi G. Takaoka2013-04-118-128/+178
| |/ / / | | | | | | | | | | | | 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 "More precise geometry checking for more keys keyboard"Tadashi G. Takaoka2013-04-111-2/+3
|\ \ \ \ \
| * | | | | More precise geometry checking for more keys keyboardTadashi G. Takaoka2013-04-111-2/+3
| | |/ / / | |/| | | | | | | | | | | | | | | | | | Bug: 8484528 Change-Id: I0dd09530a5262fee9c19bb0f11d5830b00306944
* | | | | 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 "Preserve a class for testing"Kurt Partridge2013-04-101-0/+3
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Preserve a class for testingKurt Partridge2013-04-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Addresses b/8583091 Change-Id: I89605223784312d9572dfc7c8aaa7e5eba44c4f6
* | | | | Better release build reportingKurt Partridge2013-04-101-4/+21
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ResearchLogger reports whether a build is a release build or not to avoid polluting data with IME debugging work by developers. Previously this was done by checking a constant flag, which was also serving the dual purpose of masking out debug code in release builds. This change introduces a heuristic to determine whether a build was created by a developer (using the package versionName), and annotating the data sent to the server appropriately. Change-Id: Icbad17c66b703cabf6d23d05e2c7c41bcceaae45
* | | | Merge "Fix a bug where a space would not be inserted before a gesture"Jean Chalard2013-04-101-1/+2
|\ \ \ \
generated by cgit v1.2.3-83-g751a (git 2.47.1) at 2025-05-06 23:50:14 +0000