From 80375649d36c3e3aeec3a266827bb2d66d95d0f7 Mon Sep 17 00:00:00 2001 From: Kurt Partridge Date: Tue, 15 Jan 2013 14:15:21 -0800 Subject: [PATCH] [Rlog81b] Log more data with handleSeparator Change-Id: I4b2ebaa2398ac2f669e1e009e00842e55979e866 --- .../android/inputmethod/latin/LatinIME.java | 1 + .../inputmethod/research/ResearchLogger.java | 18 +++++++++++++++++- 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/java/src/com/android/inputmethod/latin/LatinIME.java b/java/src/com/android/inputmethod/latin/LatinIME.java index d02c4df7e..429992992 100644 --- a/java/src/com/android/inputmethod/latin/LatinIME.java +++ b/java/src/com/android/inputmethod/latin/LatinIME.java @@ -1861,6 +1861,7 @@ public final class LatinIME extends InputMethodService implements KeyboardAction final int spaceState) { if (ProductionFlag.IS_EXPERIMENTAL) { ResearchLogger.recordTimeForLogUnitSplit(); + ResearchLogger.latinIME_handleSeparator(primaryCode, mWordComposer.isComposingWord()); } boolean didAutoCorrect = false; // Handle separator diff --git a/java/src/com/android/inputmethod/research/ResearchLogger.java b/java/src/com/android/inputmethod/research/ResearchLogger.java index a2bcf4441..5822b9514 100644 --- a/java/src/com/android/inputmethod/research/ResearchLogger.java +++ b/java/src/com/android/inputmethod/research/ResearchLogger.java @@ -1719,7 +1719,7 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang /** * Record the current time in case the LogUnit is later split. * - * If the current logUnitis split, then tapping, motion events, etc. before this time should + * If the current logUnit is split, then tapping, motion events, etc. before this time should * be assigned to one LogUnit, and events after this time should go into the following LogUnit. */ public static void recordTimeForLogUnitSplit() { @@ -1728,6 +1728,22 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang researchLogger.mSavedDownEventTime = Long.MAX_VALUE; } + /** + * Log a call to LatinIME.handleSeparator() + * + * SystemResponse: The system is inserting a separator character, possibly performing auto- + * correction or other actions appropriate at the end of a word. + */ + private static final LogStatement LOGSTATEMENT_LATINIME_HANDLESEPARATOR = + new LogStatement("LatinIMEHandleSeparator", false, false, "primaryCode", + "isComposingWord"); + public static void latinIME_handleSeparator(final int primaryCode, + final boolean isComposingWord) { + final ResearchLogger researchLogger = getInstance(); + researchLogger.enqueueEvent(LOGSTATEMENT_LATINIME_HANDLESEPARATOR, primaryCode, + isComposingWord); + } + /** * Log statistics. *