| Commit message (Expand) | Author | Age | Files | Lines |
* | If there are no suggestion span, recompute suggestions.•••Bug: 8084810
Change-Id: I1743c09c43ca6835bb2f607684b037bf17d36335
|  Jean Chalard | 2013-04-15 | 3 | -16/+88 |
* | Merge "Reject a previously user-refused suggestion." |  Jean Chalard | 2013-04-15 | 3 | -0/+27 |
|\ |
|
| * | Reject a previously user-refused suggestion.•••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
|  Jean Chalard | 2013-04-15 | 3 | -0/+27 |
* | | Remove unnecessary harmful finish() from SettingsActivity•••Bug: 8560722
Change-Id: I825b135bd13aca17e712330660978b834648dbc3
|  Tadashi G. Takaoka | 2013-04-15 | 1 | -7/+2 |
* | | Merge "Ignore modifier key while sliding key input" |  Tadashi G. Takaoka | 2013-04-15 | 1 | -3/+3 |
|\ \ |
|
| * | | Ignore modifier key while sliding key input•••Bug: 8284624
Change-Id: I839d7a769d98f29be647dbe511ba3f73ca44446d
|  Tadashi G. Takaoka | 2013-04-15 | 1 | -3/+3 |
* | | | Merge "Suppress calling onPressKey and onReleaseKey while detecting gesture" |  Tadashi G. Takaoka | 2013-04-15 | 1 | -2/+2 |
|\ \ \ |
|
| * | | | Suppress calling onPressKey and onReleaseKey while detecting gesture•••Bug: 8284624
Change-Id: I14470bb2373965aefa921b2c94948450bfa438e3
|  Tadashi G. Takaoka | 2013-04-15 | 1 | -2/+2 |
* | | | | Add preference dependency to gesture input related options•••Bug: 8613600
Change-Id: I279d3264080aede59203205c45763c7f8ff08ae6
|  Tadashi G. Takaoka | 2013-04-15 | 1 | -4/+0 |
| |/ /
|/| | |
|
* | | | Merge "Reset vertical correction when more suggestions are in modal mode" |  Tadashi G. Takaoka | 2013-04-15 | 3 | -3/+9 |
|\ \ \ |
|
| * | | | Reset vertical correction when more suggestions are in modal mode•••Bug: 8605150
Change-Id: Iadbebfb80e765c97e58e8524eb87419f2a9e41d4
|  Tadashi G. Takaoka | 2013-04-15 | 3 | -3/+9 |
| |/ / |
|
* | | | Merge "Fix Binary dict tests" |  Jean Chalard | 2013-04-15 | 1 | -4/+4 |
|\ \ \
| |/ /
|/| | |
|
| * | | Fix Binary dict tests•••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
|  Jean Chalard | 2013-04-15 | 1 | -4/+4 |
| |/ |
|
* | | Merge "Avoid NPE" |  Kurt Partridge | 2013-04-12 | 1 | -3/+3 |
|\ \
| |/
|/| |
|
| * | Avoid NPE•••Initialize FeedbackLog earlier to avoid NPE
Change-Id: I646944c68a7d092345def16060b2a4edd8c7dbef
|  Kurt Partridge | 2013-03-26 | 1 | -3/+3 |
* | | Merge "Remove the dialog to insert words to the dictionary." |  Jean Chalard | 2013-04-12 | 3 | -22/+63 |
|\ \ |
|
| * | | Remove the dialog to insert words to the dictionary.•••Bug: 8562529
Change-Id: Ibd853f0cb995e7fb35f8730eafcef73ead58c869
|  Jean Chalard | 2013-04-12 | 3 | -22/+63 |
* | | | Merge "Restart suggestions when the cursor moves." |  Jean Chalard | 2013-04-12 | 7 | -15/+103 |
|\ \ \ |
|
| * | | | Restart suggestions when the cursor moves.•••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
|  Jean Chalard | 2013-04-12 | 7 | -15/+103 |
* | | | | Fix a bug with bad application completions•••Bug: 8561056
Change-Id: I91fb0ac76dabfc841bafc0e2c0ea9f63e6f604f9
|  Jean Chalard | 2013-04-12 | 3 | -2/+48 |
|/ / / |
|
* | | | Merge "Make shortcut-only user dict entry non-words." |  Jean Chalard | 2013-04-12 | 3 | -6/+8 |
|\ \ \ |
|
| * | | | Make shortcut-only user dict entry non-words.•••Bug: 7369456
Change-Id: Iecd6f6d617f31bd2aa6749516711aeb1f0549622
|  Jean Chalard | 2013-04-12 | 3 | -6/+8 |
| |/ / |
|
* | | | Merge "Add a class to encapsulate getting the metadata URI" |  Jean Chalard | 2013-04-12 | 2 | -1/+29 |
|\ \ \
| |/ /
|/| | |
|
| * | | Add a class to encapsulate getting the metadata URI•••Bug: 8376173
Change-Id: Icaac8ad412e0180c44e6a955943e3b208981b2ea
|  Jean Chalard | 2013-04-11 | 2 | -1/+29 |
* | | | Merge "Better isolate ResearchLogging data" |  Kurt Partridge | 2013-04-11 | 1 | -0/+3 |
|\ \ \ |
|
| * | | | Better isolate ResearchLogging data•••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
|  Kurt Partridge | 2013-04-10 | 1 | -0/+3 |
* | | | | Merge "Add some utility functions." |  Jean Chalard | 2013-04-11 | 1 | -0/+71 |
|\ \ \ \ |
|
| * | | | | Add some utility functions.•••Bug: 7657025
Change-Id: I4889721b5348c77ed56c5157557e9988dea48a02
|  Jean Chalard | 2013-04-11 | 1 | -0/+71 |
| | |/ /
| |/| | |
|
* | | | | Add debug message to detect unusual zero width condition•••Bug: 8452013
Change-Id: I0169dc6b0d08d1b4d9e01a5fa2ad01e9db292bda
|  Tadashi G. Takaoka | 2013-04-11 | 1 | -0/+12 |
* | | | | Merge "Add TypefaceUtils" |  Tadashi G. Takaoka | 2013-04-11 | 8 | -128/+178 |
|\ \ \ \ |
|
| * | | | | Add TypefaceUtils•••Change-Id: I2d939bc1e3279253902b28ff12bbf7861716b64d
|  Tadashi G. Takaoka | 2013-04-11 | 8 | -128/+178 |
| |/ / / |
|
* | | | | Merge "Use SparseIntArray instead of TreeMap<Integer, Integer>" |  Tadashi G. Takaoka | 2013-04-11 | 1 | -42/+24 |
|\ \ \ \ |
|
| * | | | | Use SparseIntArray instead of TreeMap<Integer, Integer>•••Change-Id: I56a64228a5fd85d440d0d08ac67cba8bf2e5690d
|  Tadashi G. Takaoka | 2013-04-10 | 1 | -42/+24 |
* | | | | | Merge "More precise geometry checking for more keys keyboard" |  Tadashi G. Takaoka | 2013-04-11 | 1 | -2/+3 |
|\ \ \ \ \ |
|
| * | | | | | More precise geometry checking for more keys keyboard•••Bug: 8484528
Change-Id: I0dd09530a5262fee9c19bb0f11d5830b00306944
|  Tadashi G. Takaoka | 2013-04-11 | 1 | -2/+3 |
| | |/ / /
| |/| | | |
|
* | | | | | Merge "Fix two bugs in dicttool" |  Jean Chalard | 2013-04-11 | 1 | -3/+6 |
|\ \ \ \ \
| |/ / / /
|/| | | | |
|
| * | | | | Fix two bugs in dicttool•••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
|  Jean Chalard | 2013-04-10 | 1 | -3/+6 |
* | | | | | Merge "Preserve a class for testing" |  Kurt Partridge | 2013-04-10 | 1 | -0/+3 |
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
|
| * | | | | Preserve a class for testing•••Addresses b/8583091
Change-Id: I89605223784312d9572dfc7c8aaa7e5eba44c4f6
|  Kurt Partridge | 2013-04-10 | 1 | -0/+3 |
* | | | | | Better release build reporting•••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
|  Kurt Partridge | 2013-04-10 | 1 | -4/+21 |
|/ / / / |
|
* | | | | Merge "Fix a bug where a space would not be inserted before a gesture" |  Jean Chalard | 2013-04-10 | 1 | -1/+2 |
|\ \ \ \ |
|
| * | | | | Fix a bug where a space would not be inserted before a gesture•••Bug: 8583675
Change-Id: I14924fef5663f48481994c02c56be45f9f85be65
|  Jean Chalard | 2013-04-10 | 1 | -1/+2 |
| |/ / / |
|
* | | | | Merge "Use private object as a lock instead of synchronized method" |  Tadashi G. Takaoka | 2013-04-10 | 1 | -123/+145 |
|\ \ \ \ |
|
| * | | | | Use private object as a lock instead of synchronized method•••Change-Id: I0b9a935d8c97ddfb18ab7950a9d6bf542863072a
|  Tadashi G. Takaoka | 2013-04-10 | 1 | -123/+145 |
| |/ / / |
|
* | | | | Merge "Use private lock object instead of synchronized method" |  Tadashi G. Takaoka | 2013-04-10 | 1 | -22/+31 |
|\ \ \ \ |
|
| * | | | | Use private lock object instead of synchronized method•••Change-Id: Ifc62bacbd0583a7d102009681a94bdd9ccff7d47
|  Tadashi G. Takaoka | 2013-04-10 | 1 | -22/+31 |
| |/ / / |
|
* | | | | Merge "Protect simultaneously modifying gesture preview data" |  Tadashi G. Takaoka | 2013-04-10 | 1 | -1/+15 |
|\ \ \ \
| |/ / /
|/| | | |
|
| * | | | Protect simultaneously modifying gesture preview data•••Bug: 8556775
Change-Id: I83272e3adbfc0c9cc14f9e8b479e926aabf4fa2a
|  Tadashi G. Takaoka | 2013-04-10 | 1 | -1/+15 |
* | | | | Fix a bug where autocorrection status would be wrong.•••Change-Id: Ic220129dc59f585164dbf63591cd1c96de17fe6f
|  Jean Chalard | 2013-04-10 | 7 | -10/+11 |
|/ / / |
|
* | | | Don't override EXTRA_SHOW_FRAGMENT if exists•••Bug: 8541190
Change-Id: I37d3de12558efd55f157389d2e0b51fc83dc5da1
|  Tadashi G. Takaoka | 2013-04-10 | 1 | -1/+4 |