Merge "Log LatinIME.commitCurrentAutoCorrection()"

main
Kurt Partridge 2012-12-23 11:47:47 -08:00 committed by Android (Google) Code Review
commit 13b1be988d
2 changed files with 26 additions and 1 deletions

View File

@ -2095,6 +2095,11 @@ public final class LatinIME extends InputMethodService implements KeyboardAction
typedWord, autoCorrection.toString(), separatorString, mWordComposer); typedWord, autoCorrection.toString(), separatorString, mWordComposer);
} }
mExpectingUpdateSelection = true; mExpectingUpdateSelection = true;
if (ProductionFlag.IS_EXPERIMENTAL) {
ResearchLogger.latinIme_commitCurrentAutoCorrection(typedWord,
autoCorrection.toString(), separatorString);
}
commitChosenWord(autoCorrection, LastComposedWord.COMMIT_TYPE_DECIDED_WORD, commitChosenWord(autoCorrection, LastComposedWord.COMMIT_TYPE_DECIDED_WORD,
separatorString); separatorString);
if (!typedWord.equals(autoCorrection)) { if (!typedWord.equals(autoCorrection)) {

View File

@ -1243,6 +1243,26 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
completionInfo); completionInfo);
} }
/**
* Log a call to LatinIME.commitCurrentAutoCorrection().
*
* SystemResponse: The IME has committed an auto-correction. An auto-correction changes the raw
* text input to another word that the user more likely desired to type.
*/
private static final LogStatement LOGSTATEMENT_LATINIME_COMMITCURRENTAUTOCORRECTION =
new LogStatement("LatinIMECommitCurrentAutoCorrection", true, false, "typedWord",
"autoCorrection", "separatorString");
public static void latinIme_commitCurrentAutoCorrection(final String typedWord,
final String autoCorrection, final String separatorString) {
final String scrubbedTypedWord = scrubDigitsFromString(typedWord);
final String scrubbedAutoCorrection = scrubDigitsFromString(autoCorrection);
final ResearchLogger researchLogger = getInstance();
researchLogger.enqueueEvent(LOGSTATEMENT_LATINIME_COMMITCURRENTAUTOCORRECTION,
scrubbedTypedWord, scrubbedAutoCorrection, separatorString);
researchLogger.onWordComplete(scrubbedAutoCorrection, Long.MAX_VALUE,
false /* isPartial */);
}
private boolean isExpectingCommitText = false; private boolean isExpectingCommitText = false;
public static void latinIME_commitPartialText(final CharSequence committedWord, public static void latinIME_commitPartialText(final CharSequence committedWord,
final long lastTimestampOfWordData) { final long lastTimestampOfWordData) {
@ -1265,9 +1285,9 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
final ResearchLogger researchLogger = getInstance(); final ResearchLogger researchLogger = getInstance();
final String scrubbedWord = scrubDigitsFromString(committedWord.toString()); final String scrubbedWord = scrubDigitsFromString(committedWord.toString());
if (!researchLogger.isExpectingCommitText) { if (!researchLogger.isExpectingCommitText) {
researchLogger.onWordComplete(scrubbedWord, Long.MAX_VALUE, false /* isPartial */);
researchLogger.enqueueEvent(LOGSTATEMENT_RICHINPUTCONNECTIONCOMMITTEXT, researchLogger.enqueueEvent(LOGSTATEMENT_RICHINPUTCONNECTIONCOMMITTEXT,
newCursorPosition); newCursorPosition);
researchLogger.onWordComplete(scrubbedWord, Long.MAX_VALUE, false /* isPartial */);
} }
researchLogger.isExpectingCommitText = false; researchLogger.isExpectingCommitText = false;
} }