Revert "[HS1] Save some data in startInputView"
This reverts commit f1accd4f04
.
Bug: 16830599
Change-Id: I4c9367f742b8ee8463920e91263b05c2195f8480
This commit is contained in:
parent
772a8019ef
commit
1f3d89c2af
1 changed files with 7 additions and 29 deletions
|
@ -752,34 +752,8 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
loadKeyboard();
|
loadKeyboard();
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* A class that holds information to pass from onStartInputInternal to onStartInputViewInternal
|
|
||||||
*
|
|
||||||
* OnStartInput needs to reload the settings and that will prevent onStartInputViewInternal
|
|
||||||
* from comparing the old settings with the new ones, so we use this memory to pass the
|
|
||||||
* necessary information along.
|
|
||||||
*/
|
|
||||||
private static class EditorChangeInfo {
|
|
||||||
public final boolean mIsSameInputType;
|
|
||||||
public final boolean mHasSameOrientation;
|
|
||||||
public EditorChangeInfo(final boolean isSameInputType, final boolean hasSameOrientation) {
|
|
||||||
mIsSameInputType = isSameInputType;
|
|
||||||
mHasSameOrientation = hasSameOrientation;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
private EditorChangeInfo mLastEditorChangeInfo;
|
|
||||||
|
|
||||||
private void onStartInputInternal(final EditorInfo editorInfo, final boolean restarting) {
|
private void onStartInputInternal(final EditorInfo editorInfo, final boolean restarting) {
|
||||||
super.onStartInput(editorInfo, restarting);
|
super.onStartInput(editorInfo, restarting);
|
||||||
if (editorInfo == null) {
|
|
||||||
Log.e(TAG, "Null EditorInfo in onStartInput()");
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
SettingsValues currentSettingsValues = mSettings.getCurrent();
|
|
||||||
mLastEditorChangeInfo = new EditorChangeInfo(
|
|
||||||
currentSettingsValues.isSameInputType(editorInfo),
|
|
||||||
currentSettingsValues.hasSameOrientation(getResources().getConfiguration()));
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@SuppressWarnings("deprecation")
|
@SuppressWarnings("deprecation")
|
||||||
|
@ -789,6 +763,9 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
final KeyboardSwitcher switcher = mKeyboardSwitcher;
|
final KeyboardSwitcher switcher = mKeyboardSwitcher;
|
||||||
switcher.updateKeyboardTheme();
|
switcher.updateKeyboardTheme();
|
||||||
final MainKeyboardView mainKeyboardView = switcher.getMainKeyboardView();
|
final MainKeyboardView mainKeyboardView = switcher.getMainKeyboardView();
|
||||||
|
// If we are starting input in a different text field from before, we'll have to reload
|
||||||
|
// settings, so currentSettingsValues can't be final.
|
||||||
|
SettingsValues currentSettingsValues = mSettings.getCurrent();
|
||||||
|
|
||||||
if (editorInfo == null) {
|
if (editorInfo == null) {
|
||||||
Log.e(TAG, "Null EditorInfo in onStartInputView()");
|
Log.e(TAG, "Null EditorInfo in onStartInputView()");
|
||||||
|
@ -831,7 +808,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
accessUtils.onStartInputViewInternal(mainKeyboardView, editorInfo, restarting);
|
accessUtils.onStartInputViewInternal(mainKeyboardView, editorInfo, restarting);
|
||||||
}
|
}
|
||||||
|
|
||||||
final boolean inputTypeChanged = !mLastEditorChangeInfo.mIsSameInputType;
|
final boolean inputTypeChanged = !currentSettingsValues.isSameInputType(editorInfo);
|
||||||
final boolean isDifferentTextField = !restarting || inputTypeChanged;
|
final boolean isDifferentTextField = !restarting || inputTypeChanged;
|
||||||
if (isDifferentTextField) {
|
if (isDifferentTextField) {
|
||||||
mSubtypeSwitcher.updateParametersOnStartInputView();
|
mSubtypeSwitcher.updateParametersOnStartInputView();
|
||||||
|
@ -876,12 +853,13 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
|
||||||
canReachInputConnection = true;
|
canReachInputConnection = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (isDifferentTextField || !mLastEditorChangeInfo.mHasSameOrientation) {
|
if (isDifferentTextField ||
|
||||||
|
!currentSettingsValues.hasSameOrientation(getResources().getConfiguration())) {
|
||||||
loadSettings();
|
loadSettings();
|
||||||
}
|
}
|
||||||
final SettingsValues currentSettingsValues = mSettings.getCurrent();
|
|
||||||
if (isDifferentTextField) {
|
if (isDifferentTextField) {
|
||||||
mainKeyboardView.closing();
|
mainKeyboardView.closing();
|
||||||
|
currentSettingsValues = mSettings.getCurrent();
|
||||||
|
|
||||||
if (currentSettingsValues.mAutoCorrectionEnabledPerUserSettings) {
|
if (currentSettingsValues.mAutoCorrectionEnabledPerUserSettings) {
|
||||||
suggest.setAutoCorrectionThreshold(
|
suggest.setAutoCorrectionThreshold(
|
||||||
|
|
Loading…
Reference in a new issue