am 5523a681
: Merge "[Rlog59a] Split logUnits correctly around separators"
* commit '5523a6818d77cd198e349f075bb1bb5acaff5b71': [Rlog59a] Split logUnits correctly around separators
This commit is contained in:
commit
988a2d0c87
2 changed files with 16 additions and 1 deletions
|
@ -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()) {
|
||||
|
|
|
@ -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.
|
||||
*
|
||||
|
|
Loading…
Reference in a new issue