aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/research (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Allow LogUnits to hold >1 wordKurt Partridge2013-04-194-86/+147
| | | | | | | | | | | | | | 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
* Merge "Fix bug in counting words between samples"Kurt Partridge2013-04-182-20/+23
|\
| * Fix bug in counting words between samplesKurt Partridge2013-04-172-20/+23
| | | | | | | | | | | | | | | | 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
* | Merge "Fix user recording dialog"Kurt Partridge2013-04-181-28/+9
|\ \
| * | Fix user recording dialogKurt Partridge2013-04-171-28/+9
| |/ | | | | | | | | | | | | | | 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
* | Merge "Fix alarm bug"Kurt Partridge2013-04-181-19/+6
|\ \
| * | Fix alarm bugKurt Partridge2013-04-171-19/+6
| |/ | | | | | | | | | | The AlarmManager takes absolute times as an argument, not durations. Change-Id: I419434ba5908c5f4070608070bbecf753088ecc8
* | Merge "Label logUnits after LatinImeOnEndBatchInput"Kurt Partridge2013-04-171-0/+3
|\ \
| * | Label logUnits after LatinImeOnEndBatchInputKurt Partridge2013-04-171-0/+3
| |/ | | | | | | | | | | | | | | | | | | | | 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
* | Merge "Fix NPE"Kurt Partridge2013-04-171-1/+1
|\ \
| * | Fix NPEKurt Partridge2013-04-161-1/+1
| |/ | | | | | | Change-Id: I8a091fb63dab12a0d2ba69e2fe393249bc68d103
* | Merge "Clean up unnecessary log messages"Kurt Partridge2013-04-171-2/+0
|\ \
| * | Clean up unnecessary log messagesKurt Partridge2013-04-161-2/+0
| |/ | | | | | | Change-Id: I6c3252f3b31c5874ea2f909f636f58b579c06b8d
* | Merge "Fix logic error"Kurt Partridge2013-04-171-1/+1
|\ \
| * | Fix logic errorKurt Partridge2013-04-161-1/+1
| |/ | | | | | | Change-Id: Ia36660c66504ba2fbb7f125704926a08726df87a
* | Merge "Fix JsonWriter bug"Kurt Partridge2013-04-171-3/+13
|\ \
| * | Fix JsonWriter bugKurt Partridge2013-04-171-3/+13
| |/ | | | | | | | | | | | | JsonWriter requires that data be written to it, else close() will throw an exception. Change-Id: I596c5363e063cc75bcda55e0a506eefb3f17bd67
* | Merge "Small dot in corner was not visible"Kurt Partridge2013-04-171-1/+1
|\ \
| * | Small dot in corner was not visibleKurt Partridge2013-04-161-1/+1
| |/ | | | | | | | | | | Framework change perhaps? Even if not, this makes it more visible Change-Id: Ib6d36b5a30136c4516adac666bdff36d1cd9b337
* / Include nanoTime in filenameKurt Partridge2013-04-162-9/+11
|/ | | | | | | System is fast enough that sometimes SystemClock.currentTimeMillis() is duplicated when used to make a unique filename. Change-Id: I9454fbb5e10265d36b8e17cba183a1591d52cc7b
* 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
* | Restart suggestions when the cursor moves.Jean Chalard2013-04-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | 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
* | 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 "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 "[FileEncap18] Clean up uploading scheduling"Kurt Partridge2013-04-093-31/+51
|\ \
| * | [FileEncap18] Clean up uploading schedulingKurt Partridge2013-03-253-31/+51
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Move scheduling logic from ResearchLogger.java to UploaderService.java - Switch to a one-shot timer. Previously the uploader was scheduled on an inexact repeating schedule. It's better to reschedule the next upload after the current one is finished to reduce the chances of multiple uploads happening at the same time. - Avoid double-execution - Previously a scheduled upload might run right after an explicit one if they occured at the same time. This change reduces the chances of this. - Some method extraction and naming Change-Id: I9efda11be77d334c7f61bd40a36d65f0421ebde4
* | Merge "Allow logging while replaying"Kurt Partridge2013-04-091-2/+1
|\ \
| * | Allow logging while replayingKurt Partridge2013-04-011-2/+1
| | | | | | | | | | | | | | | | | | | | | Previously logging was disabled during replay. This makes it impossible to use logged data as a regression test, since the new log was unavailable. This change corrects this problem. Change-Id: I19dc31def2f2f87fd219dc561c739d18e4ab9c9c
* | | Merge "Split LogUnits differently."Kurt Partridge2013-04-011-2/+1
|\ \ \ | |/ / |/| |
| * | Split LogUnits differently.Kurt Partridge2013-03-261-2/+1
| |/ | | | | | | | | | | | | | | | | Previously an autocorrection caused a new LogUnit to be started, splitting off the previous LogUnit right at the autocorrection method time. This change causes the split to happen before the MotionEvents that led to the autocorrection being called. Change-Id: I2504df8eb47ee77e5f46bac34a8450636c03fd9f
* / Close ResearchLogger upon onFinishInputViewKurt Partridge2013-03-251-12/+14
|/ | | | | | | | | | | | Previously, ResearchLogger#stop() was called both in LatinIME#onFinishInputView() and in LatinIME#onWindowHidden(). This resulted in multiple logs being written. Since onFinishInputView is the more reliable of the two (it is called in InputMethodService#onDestroy; onWindowHidden is not), the code now uses onFinishInputView as a stopping signal. Change-Id: Iae4b8c3bdab226027624eeab19b3737367e4a108
* resolved conflicts for merge of 5b048292 to masterTadashi G. Takaoka2013-03-1910-20/+32
|\ | | | | | | Change-Id: I67ebab46954cd7b8e3e79e7fed523bafb15f8835
| * Rename ProductionFlag.IS_EXPERIMENTAL to USES_DEVELOPMENT_ONLY_DIAGNOSTICSKen Wakasa2013-03-189-19/+30
| | | | | | | | | | bug: 8393568 Change-Id: Ie5edf44a3627aca9416145aff56bf05bbf2a05f3
* | [FileEncap9] Extract ResearchLogDirectory classKurt Partridge2013-03-145-82/+153
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously used a monotonically increasing int. Now uses uuid and nanoseconds. squashed in: [FileEncap11] Read preference from ResearchSettings Change-Id: Ic779e0a69db6b16e92c6f4b63dbe7b7add566ab6 [FileEncap12] Simplify directory cleanup invocation Change-Id: I688047409c0343d32b11447fb625dfb726c731ec [FileEncap14] Change log filename syntax Change-Id: I9243b20b2eb392f81ab8c5c3d19315211240e0bc Change-Id: I5c9d70e0cb7b0965158e17dd71dfab796bd9a440
* | Merge "[Lazy4] Remove useless debug code"Kurt Partridge2013-03-141-33/+0
|\ \
| * | [Lazy4] Remove useless debug codeJean Chalard2013-03-081-33/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | This code is cluttering readability and getting in the way of future updates. Future updates will make it much easier to debug and to test this code, so this needs to disappear. Change-Id: I32e28ec608587b6b7c07250a2692b13e8fc98465
* | | [Lazy2] Pass a runnable to abortJean Chalard2013-03-121-4/+11
| | | | | | | | | | | | Change-Id: I721afbf426099753aa876a55323eb806b08c24d6
* | | [Lazy1] Switch to blocking log closuresJean Chalard2013-03-122-42/+45
|/ / | | | | | | Change-Id: I4daec20b7b47b0d71c5aab6e17cd660015e19e71
* | [FileEncap8] Remove useless "success" variableKurt Partridge2013-03-071-13/+3
| | | | | | | | Change-Id: Icf76bc0b795b6a9221d9a6d61c38df31af54ab7b
* | [FileEncap7] Extract uploadContents methodKurt Partridge2013-03-071-8/+11
| | | | | | | | Change-Id: I018b2726533042dbcb25713ea039aec41c148db2
* | [FileEncap6] Extract Uploader classKurt Partridge2013-03-072-162/+197
| | | | | | | | | | | | Gerrit was stuck on original change I9ef4eb7450 Change-Id: I6d4ee686944074779aca5b09b018782c7033874d
* | [FileEncap5] Move conditional logic to callerKurt Partridge2013-03-061-3/+4
| | | | | | | | Change-Id: I6599539059f86dc8902210d325439c08035b2e2a
* | [FileEncap4] Simplify logicKurt Partridge2013-03-061-4/+5
| | | | | | | | Change-Id: Ibb8660d12812b867856259d3e5c04a31e281027d
* | [FileEncap3] Extract isUploadingUnconditionally methodKurt Partridge2013-03-061-5/+8
| | | | | | | | | | | | - remove isUploadingUnconditionally local variable Change-Id: I69be8708c1f306568cbb9c5cbefc9e42a87df497
* | [FileEncap2] Remove mCanUpload instance variableKurt Partridge2013-03-061-6/+5
| | | | | | | | Change-Id: I62a100c4d73bb64a14a7994f7dd9af7b6946d064
* | [FileEncap1] Extract hasUploadingPermisssions methodKurt Partridge2013-03-061-4/+7
| | | | | | | | Change-Id: Ica1dabcf066cde6580035f12fa35a404cb32fb3f
generated by cgit v1.2.3-83-g751a (git 2.47.1) at 2025-05-09 23:21:46 +0000