Cleanup (A57)

We stopped cancelling manual picks a few weeks ago. This code
is dead.

Change-Id: I4032fcc3c95e9379f1839fe860a1b8a9bd7d0bc7
This commit is contained in:
Jean Chalard 2012-07-05 11:43:03 +09:00
parent 29eeef75ec
commit 0e9e7e337d
3 changed files with 31 additions and 18 deletions

View File

@ -73,10 +73,10 @@ public class LastComposedWord {
} }
public boolean canRevertCommit() { public boolean canRevertCommit() {
return mActive && !TextUtils.isEmpty(mCommittedWord); return mActive && !TextUtils.isEmpty(mCommittedWord) && !didCommitTypedWord();
} }
public boolean didCommitTypedWord() { private boolean didCommitTypedWord() {
return TextUtils.equals(mTypedWord, mCommittedWord); return TextUtils.equals(mTypedWord, mCommittedWord);
} }

View File

@ -2000,12 +2000,6 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
mUserHistoryDictionary.cancelAddingUserHistory( mUserHistoryDictionary.cancelAddingUserHistory(
previousWord.toString(), committedWord.toString()); previousWord.toString(), committedWord.toString());
} }
if (0 == separatorLength || mLastComposedWord.didCommitTypedWord()) {
// This is the case when we cancel a manual pick.
// We should restart suggestion on the word right away.
mWordComposer.resumeSuggestionOnLastComposedWord(mLastComposedWord);
mConnection.setComposingText(originallyTypedWord, 1);
} else {
mConnection.commitText(originallyTypedWord, 1); mConnection.commitText(originallyTypedWord, 1);
// Re-insert the separator // Re-insert the separator
sendKeyCodePoint(mLastComposedWord.mSeparatorCode); sendKeyCodePoint(mLastComposedWord.mSeparatorCode);
@ -2016,7 +2010,6 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
} }
// Don't restart suggestion yet. We'll restart if the user deletes the // Don't restart suggestion yet. We'll restart if the user deletes the
// separator. // separator.
}
mLastComposedWord = LastComposedWord.NOT_A_COMPOSED_WORD; mLastComposedWord = LastComposedWord.NOT_A_COMPOSED_WORD;
mHandler.postUpdateSuggestions(); mHandler.postUpdateSuggestions();
} }

View File

@ -126,6 +126,26 @@ public class InputLogicTests extends InputTestsBase {
mTextView.getText().toString()); mTextView.getText().toString());
} }
public void testAutoCorrectWithSpaceThenRevert() {
final String STRING_TO_TYPE = "tgis ";
final String EXPECTED_RESULT = "tgis ";
type(STRING_TO_TYPE);
mLatinIME.onUpdateSelection(0, 0, STRING_TO_TYPE.length(), STRING_TO_TYPE.length(), -1, -1);
type(Keyboard.CODE_DELETE);
assertEquals("auto-correct with space then revert", EXPECTED_RESULT,
mTextView.getText().toString());
}
public void testAutoCorrectToSelfDoesNotRevert() {
final String STRING_TO_TYPE = "this ";
final String EXPECTED_RESULT = "this";
type(STRING_TO_TYPE);
mLatinIME.onUpdateSelection(0, 0, STRING_TO_TYPE.length(), STRING_TO_TYPE.length(), -1, -1);
type(Keyboard.CODE_DELETE);
assertEquals("auto-correct with space does not revert", EXPECTED_RESULT,
mTextView.getText().toString());
}
public void testDoubleSpace() { public void testDoubleSpace() {
final String STRING_TO_TYPE = "this "; final String STRING_TO_TYPE = "this ";
final String EXPECTED_RESULT = "this. "; final String EXPECTED_RESULT = "this. ";