Move some code inside a function to simplify flow
Also rename the function to reflect better the behavior. An upcoming change will clean up the remnants Change-Id: I19bdf0d9e95f62ee749e1f64fe17132c4a5ef0bc
This commit is contained in:
parent
46798d8f1f
commit
1c6cf26c37
1 changed files with 8 additions and 12 deletions
|
@ -1602,17 +1602,7 @@ public class LatinIME extends InputMethodServiceCompatWrapper implements Keyboar
|
||||||
final boolean shouldAutoCorrect = mSettingsValues.mAutoCorrectEnabled
|
final boolean shouldAutoCorrect = mSettingsValues.mAutoCorrectEnabled
|
||||||
&& !mInputTypeNoAutoCorrect;
|
&& !mInputTypeNoAutoCorrect;
|
||||||
if (shouldAutoCorrect && primaryCode != Keyboard.CODE_SINGLE_QUOTE) {
|
if (shouldAutoCorrect && primaryCode != Keyboard.CODE_SINGLE_QUOTE) {
|
||||||
final boolean pickedDefaultSuggestion = pickDefaultSuggestion(primaryCode);
|
final boolean didAutoCorrect = commitCurrentAutoCorrection(primaryCode, ic);
|
||||||
if (pickedDefaultSuggestion) {
|
|
||||||
final CharSequence autoCorrection = mWordComposer.getAutoCorrectionOrNull();
|
|
||||||
final String typedWord = mWordComposer.getTypedWord();
|
|
||||||
if (!typedWord.equals(autoCorrection)) {
|
|
||||||
// This will make the correction flash for a short while as a visual clue
|
|
||||||
// to the user that auto-correction happened.
|
|
||||||
InputConnectionCompatUtils.commitCorrection(ic,
|
|
||||||
mLastSelectionEnd - typedWord.length(), typedWord, autoCorrection);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
} else {
|
} else {
|
||||||
commitTyped(ic);
|
commitTyped(ic);
|
||||||
}
|
}
|
||||||
|
@ -1869,7 +1859,7 @@ public class LatinIME extends InputMethodServiceCompatWrapper implements Keyboar
|
||||||
setSuggestionStripShown(isSuggestionsStripVisible());
|
setSuggestionStripShown(isSuggestionsStripVisible());
|
||||||
}
|
}
|
||||||
|
|
||||||
private boolean pickDefaultSuggestion(int separatorCode) {
|
private boolean commitCurrentAutoCorrection(final int separatorCode, final InputConnection ic) {
|
||||||
// Complete any pending suggestions query first
|
// Complete any pending suggestions query first
|
||||||
if (mHandler.hasPendingUpdateSuggestions()) {
|
if (mHandler.hasPendingUpdateSuggestions()) {
|
||||||
mHandler.cancelUpdateSuggestions();
|
mHandler.cancelUpdateSuggestions();
|
||||||
|
@ -1891,6 +1881,12 @@ public class LatinIME extends InputMethodServiceCompatWrapper implements Keyboar
|
||||||
// Add the word to the user unigram dictionary if it's not a known word
|
// Add the word to the user unigram dictionary if it's not a known word
|
||||||
addToUserUnigramAndBigramDictionaries(autoCorrection,
|
addToUserUnigramAndBigramDictionaries(autoCorrection,
|
||||||
UserUnigramDictionary.FREQUENCY_FOR_TYPED);
|
UserUnigramDictionary.FREQUENCY_FOR_TYPED);
|
||||||
|
if (!typedWord.equals(autoCorrection)) {
|
||||||
|
// This will make the correction flash for a short while as a visual clue
|
||||||
|
// to the user that auto-correction happened.
|
||||||
|
InputConnectionCompatUtils.commitCorrection(ic,
|
||||||
|
mLastSelectionEnd - typedWord.length(), typedWord, autoCorrection);
|
||||||
|
}
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
return false;
|
return false;
|
||||||
|
|
Loading…
Reference in a new issue