nyancrimew
/
LatinIME
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge "[Rlog59a] Split logUnits correctly around separators"

main
Kurt Partridge 2013-01-11 09:18:10 -08:00 committed by Android (Google) Code Review
parent f6870cc82d 90aaa2d0a5
commit 5523a6818d
2 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
java/src/com/android/inputmethod/latin/LatinIME.java
View File

@ -1843,6 +1843,9 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
// Returns true if we did an autocorrection, false otherwise.
private boolean handleSeparator(final int primaryCode, final int x, final int y,
final int spaceState) {
if (ProductionFlag.IS_EXPERIMENTAL) {
ResearchLogger.latinIME_handleSeparator();
}
boolean didAutoCorrect = false;
// Handle separator
if (mWordComposer.isComposingWord()) {

14
java/src/com/android/inputmethod/research/ResearchLogger.java
View File

@ -838,6 +838,10 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
mCurrentLogUnit = newLogUnit;
}
private void setSavedDownEventTime(final long time) {
mSavedDownEventTime = time;
}
public void onWordFinished(final String word, final boolean isBatchMode) {
commitCurrentLogUnitAsWord(word, mSavedDownEventTime, isBatchMode);
mSavedDownEventTime = Long.MAX_VALUE;
@ -987,7 +991,7 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
if (action == MotionEvent.ACTION_DOWN) {
// Subtract 1 from eventTime so the down event is included in the later
// LogUnit, not the earlier (the test is for inequality).
researchLogger.mSavedDownEventTime = eventTime - 1;
researchLogger.setSavedDownEventTime(eventTime - 1);
}
}
}
@ -1637,6 +1641,14 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
researchLogger.mStatistics.recordGestureDelete();
}
public static void latinIME_handleSeparator() {
// Reset the saved down event time. For tapping, motion events, etc. before the separator
// are assigned to the previous LogUnit, and events after the separator are assigned to the
// next LogUnit. In the case of multitap, this might capture down events corresponding to
// the next word, however it should not be more than a character or two.
getInstance().setSavedDownEventTime(SystemClock.uptimeMillis());
}
/**
* Log statistics.
*