aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/research (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Fix buildTadashi G. Takaoka2013-08-261-1/+1
| | | | Change-Id: I77e25fd4032b7fc3b4b565c9bab18a2824d7bb76
* [AC2] Reference a dict rather than a string in suggestion infosJean Chalard2013-08-201-1/+1
| | | | | Bug: 9059617 Change-Id: Ic17bc0fd5d812268fd37d7fd35b4e9ebfb95fa5e
* Add getters and copy constructor to Key classTadashi G. Takaoka2013-08-151-5/+5
| | | | Change-Id: I78eb96dcc26378926dc63acbfeb4828cbd44435d
* Remove orientation member from KeyboardIdTadashi G. Takaoka2013-08-071-2/+3
| | | | Change-Id: I2c16134a24088dbc30b74ae7c5bc578a9d84a960
* Remove CODE_RESEARCHTadashi G. Takaoka2013-08-051-7/+4
| | | | | | | Instead of CODE_RESEARCH, special outputText ".research." is used to invoke a research feedback dialog. Change-Id: Ic95c956ab4f2b5f0c3f2205c9bbcdcfb67db5d6e
* Clean up ResearchLogger log methodTadashi G. Takaoka2013-08-011-16/+18
| | | | Change-Id: I06c30aab462ce5b17157ff71d5d76f9755178080
* Fix some warningsJean Chalard2013-07-313-21/+16
| | | | Change-Id: I7290cd1fb675a1b85b9b6ac2d464c932b5bca1dd
* Merge "Do not output string-encoded MotionEvent."Kurt Partridge2013-07-151-1/+1
|\
| * Do not output string-encoded MotionEvent.Kurt Partridge2013-07-031-1/+1
| | | | | | | | | | | | | | JsonUtils knows how to output MotionEvents into json; that mechanism should be reused for SuddenJumpingTouchEventHandler. Change-Id: Id4c36c05533a3a329bf43ba33d57c0830dc4d196
* | Use TimeUnit instead of DateUtils for readabilityTadashi G. Takaoka2013-07-054-13/+14
| | | | | | | | | | Bug: 9550800 Change-Id: I087205530a5dbcff4bf08f48f4aa7068aae93215
* | Move Range out of RichInputConnection and rename it.Jean Chalard2013-07-011-2/+2
|/ | | | | | Range has grown a bit too big to stay an internal class. Change-Id: Ic3fbc48a5d10776eedd7559c4d696e5926b88288
* Merge "Record number of words entered"Kurt Partridge2013-06-281-16/+10
|\
| * Record number of words enteredKurt Partridge2013-06-271-16/+10
| | | | | | | | | | | | | | | | This change records the number of words entered in every LogUnit. This metric is helpful for determining how much multi-word gestures and spaceless tapping is used. Change-Id: I4c6d0f9d78e4ac2dd63fd53ed2ec70b368366f15
* | Fix some warningsJean Chalard2013-06-274-7/+0
|/ | | | | | Mostly remove unused imports Change-Id: If8ceffdfad79b78194094527759f5c334746973e
* Merge "Move splash screen to separate file"Kurt Partridge2013-06-242-64/+127
|\
| * Move splash screen to separate fileKurt Partridge2013-06-222-64/+127
| | | | | | | | Change-Id: I5cad5d5814303d7ca4abadd3fca1c37bd1bc357e
* | Move util classes to the latin/utils directoryKen Wakasa2013-06-241-1/+1
|/ | | | Change-Id: I1c5b27c8edf231680edb8d96f63b9d04cfc6a6fa
* Log score, kind, and sourceDict for manual suggestion and gestureKurt Partridge2013-06-132-7/+13
| | | | | | b/9422713 Change-Id: I4880fad6a5b10d09ca11b63311f14862bee724e5
* Merge "Adjust settings for development-only versions"Kurt Partridge2013-06-121-0/+6
|\
| * Adjust settings for development-only versionsKurt Partridge2013-06-121-0/+6
| | | | | | | | Change-Id: I4f08632d97f182d312edc82bda3c3897a71099ad
* | Fix ResearchLogger feedback bugKurt Partridge2013-06-121-2/+9
|/ | | | | | | | | | | | | If the logger feedback screen is up and the user hits the home button or the keyboard otherwise closes, the system won't let the user enter feedback again, because it is waiting for the user to finish the first one. With this change, the state associated with the feedback screen is cleared when the user closes the keyboard. b/9398772 Change-Id: I832a82793ad42e31b6b3fdb04fabcea75facde7d
* Avoid an NPEKurt Partridge2013-06-031-10/+10
| | | | Change-Id: I174eb8e92820fe6e7f8d3b89afae12d94063ef5d
* Merge "Fix revert of committed words"Kurt Partridge2013-06-031-13/+33
|\
| * Fix revert of committed wordsKurt Partridge2013-06-021-13/+33
| | | | | | | | | | | | | | | | | | Now that separators are put into their own LogUnits, they must be handled when going through a revert. Bug: 9088919 Change-Id: Ibebd0752bb2fa38d74ac96001d63070dd419cee3
* | Merge "Uncommit an extra LogUnit"Kurt Partridge2013-06-031-1/+6
|\ \
| * | Uncommit an extra LogUnitKurt Partridge2013-06-021-1/+6
| |/ | | | | | | | | | | | | | | | | The period-generating double-space adds an extra LogUnit -- it must be removed when reverting. Bug: 9088919 Change-Id: Ic148f40b4030a9b4a0651029bda87f7b94a52252
* | Merge "Compare scrubbed strings when uncommitting LogUnits"Kurt Partridge2013-06-031-1/+4
|\ \
| * | Compare scrubbed strings when uncommitting LogUnitsKurt Partridge2013-06-021-1/+4
| |/ | | | | | | | | | | | | | | | | | | | | | | A LogUnit is only uncommitted if the LogUnit's word matches what is expected. But a LogUnit never stores numbers, only scrubbed words that replace numbers with a special character. So when uncommitting, the text from the TextView must also be scrubbed for the comparison to pass correctly. Bug: 9088919 Change-Id: I9b56f10afce6d0cc84eb9ead3b9a9b1e061ae39c
* | Merge "Uncommit separator LogUnits"Kurt Partridge2013-06-031-3/+14
|\ \
| * | Uncommit separator LogUnitsKurt Partridge2013-06-021-3/+14
| |/ | | | | | | | | | | | | | | | | Now that separators have their own LogUnits, they must be uncommitted from the LogBuffer when backspacing over them. Bug: 9088919 Change-Id: Ib36cc94939b93abe18850a06bced17caf8aaa5b9
* / Uncommit LogUnits when reverting textInputKurt Partridge2013-06-011-0/+10
|/ | | | | | Bug: 9088919 Change-Id: I17187aa0afc1c3e9d32269c65f66cfb3a08663cb
* Merge "Remove unused method"Kurt Partridge2013-05-291-19/+0
|\
| * Remove unused methodKurt Partridge2013-05-281-19/+0
| | | | | | | | | | | | The removed method was used in old versions of ALK, never in master. Change-Id: I607009d4fcca7c7c8493fee51ecd923e6a6b3890
* | Rename containsCorrection to containsUserDeletionsKurt Partridge2013-05-282-14/+14
|/ | | | | | | This change renames containsCorrection to avoid confusion with CorrectionInfo, and to better clarify its meaning. Change-Id: I5d8f8e2b03133c15a60d4e8b0f395cda1fff5447
* Log reasons for punting n-gram loggingKurt Partridge2013-05-233-19/+107
| | | | | | Addresses b/9074239 Change-Id: I91a3bfcbd32b03e4891ff5f65be01383a3fb8975
* Merge "Always record word boundaries"Kurt Partridge2013-05-221-2/+14
|\
| * Always record word boundariesKurt Partridge2013-05-211-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | Word boundaries are currently logged, but only if all the details of a word pass the privacy filter and are logged. This change records when a word is committed in all cases, but does not disclose the word contents or any data used in its construction. Addresses b/9070768 Change-Id: I573679d0685c088aca65af99e46337a2f429f816
* | Modify isAllowedToLogKurt Partridge2013-05-213-6/+71
|/ | | | | | | | | | | | | | | | Currently isAllowedToLog only checks the state of the ResearchLogger, and does not consider which log the data is going to. This causes problems with the logs for Feedback. The Feedback log should inhibit normal logging procedures, but the system needs to be able to write specific data directly to it. This change renames to isAllowedToLogTo and adds the destination ResearchLog as a parameter. A FeedbackLog is also added as a new class so it can be distinguished from other ResearchLogs. Change-Id: I5a1eea05bb040c26bf816b89179f44b3024fa2ad
* Fix punctuation loggingKurt Partridge2013-05-171-10/+36
| | | | | | | | | | | | | | | | | | Punctuation is currently put into the same LogUnit as the following word. This is counter-intuitive and makes post-processing more complicated. This change causes punctuation to be put into its own LogUnit. A tricky case is the phantom space. A phantom space is inserted right before other text, whose MotionEvents have already been collected and inserted into mCurrentLogBuffer. Phantom spaces are handled by creating their own LogUnit and inserting it into the LogBuffer right before mCurrentLogBuffer is shifted in. Change-Id: I102d7cd93982ad37ba0abb6c1ac0f66b85b5f95d
* Merge "Remove unnecessary commit/uncommit"Kurt Partridge2013-05-151-1/+0
|\
| * Remove unnecessary commit/uncommitKurt Partridge2013-05-141-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the user reverts a commit, this causes the committed word to be changed. The user usually then goes back to edit that word. Going back to edit causes an uncommit, so that the editing actions are added onto the LogUnit containing the operations used to construct the word. But currently, reverting also involves its own commit/uncommit pair, with the caller performing the uncommit, and the commit happening inside the revert command. This may have been necessary in the past, when revert was called in different situations, but is unnecessary now. Furthermore, the guarding conditions are currently such that the uncommit doesn't always undo the effects of the commit, corrupting the log output. Change-Id: I74af41f4f1db2fcabfa496dcc4a2d7bd0af19b3a
* | Merge "Add claifying comment"Kurt Partridge2013-05-151-0/+16
|\ \ | |/ |/|
| * Add claifying commentKurt Partridge2013-05-141-0/+16
| | | | | | | | | | | | This is a follow-up to I7c01c3dd3ac33d7e96c00836256bae9c14b124ed Change-Id: I9e3f8968c4bbf9525d5dfe101f71373c42f88361
* | Merge "Rename a var to match semantics"Kurt Partridge2013-05-141-2/+2
|\ \
| * | Rename a var to match semanticsKurt Partridge2013-05-131-2/+2
| |/ | | | | | | | | | | | | The IS_INHIBITING_AUTO_UPLOAD flag is tested in isPossibleToUpload, so it inhibits all uploads, not just automatic ones. Renaming to IS_INHIBITING_UPLOAD. Change-Id: Iceed79260d4b062147f40c09f0c5bb06b2519b1b
* / Remove FeedbackLog instance variables.Kurt Partridge2013-05-131-42/+7
|/ | | | | | | | | | | | | | | | | For a while, the ResearchLogger supported comprehensive recording of the last few words in case the user wished to upload this data to give developers feedback. However, this mechanism was not implemented well (keeping a separate copy of the data), did not make the collected data clear to the user, and was not working properly in the latest build (NPEs). The mechanism in fact was deprecated in favor of the "userRecording" mechanism in which the user explicitly turned on comprehensive recording, demonstrated the issue they were experiencing, and then turned it off. This change removes the vestiges of the FeedbackLogging mechanism. A feedback log is still used to hold the user's message, but it is created in a local variable as-needed, and not persisted in ResearchLogger. Change-Id: I6a09d0ed36e1eddb6e57895e7428d22107f75f49
* Avoid JsonWriter multi-write errorKurt Partridge2013-05-104-49/+55
| | | | | | | | | | | JsonWriter requires that its clients pass it only a single top-level object. The existing implementation tries to make code cleaner by having mJsonWriter never be null, and instead use a global static "NULL_JSON_WRITER" that just discards data. But because JsonWriter complains if more than one top-level object is passed, making this a global object does not work. This change instead copes with mJsonWriter being null. Change-Id: Ia37ccfc8646e91f11a64713dd92d2846eb86ac54
* Merge "Remove unused code"Kurt Partridge2013-05-101-11/+1
|\
| * Remove unused codeKurt Partridge2013-05-091-11/+1
| | | | | | | | Change-Id: I192c67e9e8de44f073abc70ec85ebcdb36356c2e
* | Merge "Remove ref to LatinImeLogger preference."Kurt Partridge2013-05-101-5/+0
|\ \