aboutsummaryrefslogtreecommitdiffstats
path: root/java/src/com/android/inputmethod/research (follow)
Commit message (Collapse)AuthorAgeFilesLines
* 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
* | Little cleanupsKurt Partridge2013-03-051-7/+4
| | | | | | | | Change-Id: I2ded3d0a29610941fac27c9b26168dd6c06ceccc
* | Clean up initialization orderingKurt Partridge2013-03-051-29/+24
| | | | | | | | | | | | This change is based on an earlier one that got stuck in Gerrit: Iab77504b Change-Id: I27ad9dfb1bbb2300bd1e61d881a6ea0e116db066
* | Clean up logic in onUserLoggingConsentKurt Partridge2013-03-042-17/+17
| | | | | | | | | | | | | | | | - Make ResearchSetting for whether the user has seen the splash screen - Inline #setLoggingAllowed, which is short and is now only called by onUserLoggingConsent Change-Id: Icdf4592777b80643807b6ccf1d3896459c503e02
* | Clean up PREF_USABILITY_STUDY_MODEKurt Partridge2013-03-042-14/+15
| | | | | | | | | | | | | | | | | | | | - Refer to a common default value in DebugSettings - Make PREF_USABILITY_STUDY_MODE independent of the ResearchLogger - ResearchLogger uses its own preference through ResearchSettings multi-project commit with Ie0df836c9d779eba484b522666ec357f4e234823 Change-Id: I88547a2f619db6e7364abbbec12f9f76855dd11a
* | Merge "Remove MainLogBuffer#setSuggest()"Kurt Partridge2013-03-042-10/+13
|\ \
| * | Remove MainLogBuffer#setSuggest()Kurt Partridge2013-03-012-10/+13
| | | | | | | | | | | | | | | | | | multi-project commit with Ia98b1406fc2ee11a96893c77ea58d800fece7624 Change-Id: I8bc3d07b83dbe7d8fc56a618de07dade7f510b13
* | | Merge "Move UUID preference reading to ResearchSettings.java"Kurt Partridge2013-03-022-19/+45
|\ \ \ | |/ / |/| |
| * | Move UUID preference reading to ResearchSettings.javaKurt Partridge2013-02-282-19/+45
| | | | | | | | | | | | Change-Id: I8157249259cf8c3218c5c82a5729f4cbc1fb4eeb
* | | [TestPrep22] Make variable package-private for testingKurt Partridge2013-02-281-1/+1
| | | | | | | | | | | | Change-Id: I902a6a40df1c672c611672f34d2f3237b0d1821b
* | | [TestPrep20] Clean up import statementKurt Partridge2013-02-281-1/+0
|/ / | | | | | | Change-Id: Ifcfec09b669d8b2012b1c960bd4b1471c7946870
* | Merge "[TestPrep18] Make class public for testing"Kurt Partridge2013-02-281-1/+1
|\ \
| * | [TestPrep18] Make class public for testingKurt Partridge2013-02-261-1/+1
| | | | | | | | | | | | Change-Id: I4b457f44dd6a5a595c61a62061ea02c251fa9ab1
* | | Merge "[TestPrep17] Allow fake dictionary for testing"Kurt Partridge2013-02-271-0/+13
|\| |
| * | [TestPrep17] Allow fake dictionary for testingKurt Partridge2013-02-261-0/+13
| | | | | | | | | | | | | | | | | | | | | Currently ResearchLog requires a full dictionary to perform privacy-related checks. This makes testing difficult. This change allows a fake dictionary to be used instead. Change-Id: Ifca5bd8647475a6b84e4324117e0faa0a35479ee
* | | Merge "Ensure Preference listener is unregistered"Kurt Partridge2013-02-271-0/+6
|\ \ \ | |/ / |/| |
| * | Ensure Preference listener is unregisteredKurt Partridge2013-02-261-0/+6
| |/ | | | | | | Change-Id: I1a4d1e7bc45fc30cb1b63ea6020f0eee6e804333
* | [TestPrep11] Only use mHasWrittenData to control JsonWriter#endArray() callKurt Partridge2013-02-261-9/+5
| | | | | | | | Change-Id: If2107e38575b009a4c372194ecd827e27bdbb2b4
* | [TestPrep10] Add some commentsKurt Partridge2013-02-261-4/+28
|/ | | | Change-Id: Ia8e0bd72b0dc09bfc256ec5d1996637554376b5f
* [TestPrep8] Narrow try/catch scopeKurt Partridge2013-02-211-10/+9
| | | | Change-Id: If15f250feb81c2ad55ce1bc7dc3dafa57de87aef
* [TestPrep7] Encapsulate JsonWriter creationKurt Partridge2013-02-211-8/+19
| | | | Change-Id: I4e97fa28dd0bb1101edc0db03f024432fee6f6fe
generated by cgit v1.2.3-83-g751a (git 2.47.1) at 2025-05-09 23:44:33 +0000