| Commit message (Expand) | Author | Age | Files | Lines |
|---|
| * | Merge "Remove unused method" |   Kurt Partridge | 2013-05-29 | 1 | -19/+0 |
| |\ |
|
| | * | Remove unused method•••The removed method was used in old versions of ALK, never in master.
Change-Id: I607009d4fcca7c7c8493fee51ecd923e6a6b3890
| Kurt Partridge | 2013-05-28 | 1 | -19/+0 |
| * | | Rename containsCorrection to containsUserDeletions•••This change renames containsCorrection to avoid confusion with
CorrectionInfo, and to better clarify its meaning.
Change-Id: I5d8f8e2b03133c15a60d4e8b0f395cda1fff5447
| Kurt Partridge | 2013-05-28 | 2 | -14/+14 |
| |/ |
|
| * | Log reasons for punting n-gram logging•••Addresses b/9074239
Change-Id: I91a3bfcbd32b03e4891ff5f65be01383a3fb8975
| Kurt Partridge | 2013-05-23 | 3 | -19/+107 |
| * | Merge "Always record word boundaries" | Kurt Partridge | 2013-05-22 | 1 | -2/+14 |
| |\ |
|
| | * | Always record word boundaries•••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
| Kurt Partridge | 2013-05-21 | 1 | -2/+14 |
| * | | Modify isAllowedToLog•••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
| Kurt Partridge | 2013-05-21 | 3 | -6/+71 |
| |/ |
|
| * | Fix punctuation logging•••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
| Kurt Partridge | 2013-05-17 | 1 | -10/+36 |
| * | Merge "Remove unnecessary commit/uncommit" | Kurt Partridge | 2013-05-15 | 1 | -1/+0 |
| |\ |
|
| | * | Remove unnecessary commit/uncommit•••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
| Kurt Partridge | 2013-05-14 | 1 | -1/+0 |
| * | | Merge "Add claifying comment" | Kurt Partridge | 2013-05-15 | 1 | -0/+16 |
| |\ \
| |/
|/| |
|
| | * | Add claifying comment•••This is a follow-up to I7c01c3dd3ac33d7e96c00836256bae9c14b124ed
Change-Id: I9e3f8968c4bbf9525d5dfe101f71373c42f88361
| Kurt Partridge | 2013-05-14 | 1 | -0/+16 |
| * | | Merge "Rename a var to match semantics" | Kurt Partridge | 2013-05-14 | 1 | -2/+2 |
| |\ \ |
|
| | * | | Rename a var to match semantics•••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
| Kurt Partridge | 2013-05-13 | 1 | -2/+2 |
| | |/ |
|
| * / | Remove FeedbackLog instance variables.•••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
| Kurt Partridge | 2013-05-13 | 1 | -42/+7 |
| |/ |
|
| * | Avoid JsonWriter multi-write error•••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
| Kurt Partridge | 2013-05-10 | 4 | -49/+55 |
| * | Merge "Remove unused code" | Kurt Partridge | 2013-05-10 | 1 | -11/+1 |
| |\ |
|
| | * | Remove unused code•••Change-Id: I192c67e9e8de44f073abc70ec85ebcdb36356c2e
| Kurt Partridge | 2013-05-09 | 1 | -11/+1 |
| * | | Merge "Remove ref to LatinImeLogger preference." | Kurt Partridge | 2013-05-10 | 1 | -5/+0 |
| |\ \ |
|
| | * | | Remove ref to LatinImeLogger preference.•••This is a bug; the ResearchLogger uses ResearchSettings.readResearchLoggerEnabledFlag() instead,
which the user cannot change. (We assume they will just switch to a different keyboard.)
Change-Id: Ib5731c9eeb2bc29d9cf8608033326674f5eb6f7b
| Kurt Partridge | 2013-05-09 | 1 | -5/+0 |
| | |/ |
|
| * | | Merge "Avoid NPE by fixing ResearchLogger initialization" | Kurt Partridge | 2013-05-10 | 1 | -93/+62 |
| |\ \
| |/
|/| |
|
| | * | Avoid NPE by fixing ResearchLogger initialization•••Previously, mMainResearchLog and mMainLogBuffer were set up
when the user moved to a new TextView, and set to null when
the user left the TextView. This change causes
mMainResearchLog, mMainLogBuffer, mFeedbackLog, and
mFeedbackLogBuffer to be non-null forever after init() is
called. start() no longer sets up these fields; instead
they are cleared and reset every time stop() is called.
Checks for null values are now removed.
The earlier code just didn't initialize these variables if
the user disabled logging, but since the new version
invariantly keeps these variables valid, we add a check for
whether the user has enabled logging in publishLogUnits().
Change-Id: Ifde3517f1cf924cfa33cda95fec24529b52b3c08
| Kurt Partridge | 2013-05-09 | 1 | -93/+62 |
| * | | Fix looping logic bugs.•••shiftOut() is getting called once too often through these for loops.
Change-Id: I9a68b49e6cc1469bcddd673ab1567e238cf192b8
| Kurt Partridge | 2013-05-07 | 2 | -24/+30 |
| |/ |
|
| * | Allow LogUnits to hold >1 word•••LogUnits have been annotated with the autocorrected words, but
until now this was assumed to be a single word without spaces.
But spaceless typing can result in spaces in the LogUnit label. With this
change, the LogUnit inspects the autocorrected text to determine how many
words were inserted, and counts them accurately.
This change corrects a privacy problem, which was that if the word sampling
algorithm chose a LogUnit that actually contained multiple words, then more
than two successive words would be included in the log.
Change-Id: I7c01c3dd3ac33d7e96c00836256bae9c14b124ed
| Kurt Partridge | 2013-04-19 | 4 | -86/+147 |
| * | Merge "Fix bug in counting words between samples" | Kurt Partridge | 2013-04-18 | 2 | -20/+23 |
| |\ |
|
| | * | Fix bug in counting words between samples•••Previously MainLogBuffer#shiftOutWords() assumed it wouldn't be called if
mNumWordsUntilSafeToSample was 0. This relaxes this assumption (which is in fact
false in the current code).
Change-Id: I8723248095e84a0d9d6f4639b4742cc7dda9716b
| Kurt Partridge | 2013-04-17 | 2 | -20/+23 |
| * | | Merge "Fix user recording dialog" | Kurt Partridge | 2013-04-18 | 1 | -28/+9 |
| |\ \ |
|
| | * | | Fix user recording dialog•••Clicking the "include recording" checkbox in the user feedback dialog did nothing.
The code was relying on the state of the checkbox, rather than keeping its own state.
Fixing this addresses the bug.
Change-Id: I559d57a4e11f869f6e6f5e5de7878f765531a203
| Kurt Partridge | 2013-04-17 | 1 | -28/+9 |
| | |/ |
|
| * | | Merge "Fix alarm bug" | Kurt Partridge | 2013-04-18 | 1 | -19/+6 |
| |\ \ |
|
| | * | | Fix alarm bug•••The AlarmManager takes absolute times as an argument, not durations.
Change-Id: I419434ba5908c5f4070608070bbecf753088ecc8
| Kurt Partridge | 2013-04-17 | 1 | -19/+6 |
| | |/ |
|
| * | | Merge "Label logUnits after LatinImeOnEndBatchInput" | Kurt Partridge | 2013-04-17 | 1 | -0/+3 |
| |\ \ |
|
| | * | | Label logUnits after LatinImeOnEndBatchInput•••Previously only a commitText would cause a LogUnit to be
labeled with the word that the data generates. In the case
of gestured text, this information is available when
LatinIME#onEndBatchInput is called. Labeling the LogUnit
at this time means that the Log will have labeled words even
if stop() is called before commit.
Change-Id: Idb2f99a9c159a1b1aa00448a2ecddeca6c351c3e
| Kurt Partridge | 2013-04-17 | 1 | -0/+3 |
| | |/ |
|
| * | | Merge "Fix NPE" | Kurt Partridge | 2013-04-17 | 1 | -1/+1 |
| |\ \ |
|
| | * | | Fix NPE•••Change-Id: I8a091fb63dab12a0d2ba69e2fe393249bc68d103
| Kurt Partridge | 2013-04-16 | 1 | -1/+1 |
| | |/ |
|
| * | | Merge "Clean up unnecessary log messages" | Kurt Partridge | 2013-04-17 | 1 | -2/+0 |
| |\ \ |
|
| | * | | Clean up unnecessary log messages•••Change-Id: I6c3252f3b31c5874ea2f909f636f58b579c06b8d
| Kurt Partridge | 2013-04-16 | 1 | -2/+0 |
| | |/ |
|
| * | | Merge "Fix logic error" | Kurt Partridge | 2013-04-17 | 1 | -1/+1 |
| |\ \ |
|
| | * | | Fix logic error•••Change-Id: Ia36660c66504ba2fbb7f125704926a08726df87a
| Kurt Partridge | 2013-04-16 | 1 | -1/+1 |
| | |/ |
|
| * | | Merge "Fix JsonWriter bug" | Kurt Partridge | 2013-04-17 | 1 | -3/+13 |
| |\ \ |
|
| | * | | Fix JsonWriter bug•••JsonWriter requires that data be written to it, else close() will throw
an exception.
Change-Id: I596c5363e063cc75bcda55e0a506eefb3f17bd67
| Kurt Partridge | 2013-04-17 | 1 | -3/+13 |
| | |/ |
|
| * | | Merge "Small dot in corner was not visible" | Kurt Partridge | 2013-04-17 | 1 | -1/+1 |
| |\ \ |
|
| | * | | Small dot in corner was not visible•••Framework change perhaps? Even if not, this makes it more visible
Change-Id: Ib6d36b5a30136c4516adac666bdff36d1cd9b337
| Kurt Partridge | 2013-04-16 | 1 | -1/+1 |
| | |/ |
|
| * / | Include nanoTime in filename•••System is fast enough that sometimes SystemClock.currentTimeMillis() is duplicated
when used to make a unique filename.
Change-Id: I9454fbb5e10265d36b8e17cba183a1591d52cc7b
| Kurt Partridge | 2013-04-16 | 2 | -9/+11 |
| |/ |
|
| * | 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 |
| * | | 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 | 1 | -1/+1 |
| * | | 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 "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 |
