am 42ecf39d: am 002e43ea: Merge "Remove irrelevant options." into jb-dev

* commit '42ecf39d04df9eb02e57cf29e63552bc04092857':
  Remove irrelevant options.
This commit is contained in:
Jean Chalard 2012-06-14 14:51:37 -07:00 committed by Android Git Automerger
commit cf6e923b18
3 changed files with 9 additions and 12 deletions

View file

@ -122,7 +122,6 @@
android:defaultValue="true" /> android:defaultValue="true" />
<CheckBoxPreference <CheckBoxPreference
android:key="next_word_prediction" android:key="next_word_prediction"
android:dependency="next_word_suggestion"
android:title="@string/bigram_prediction" android:title="@string/bigram_prediction"
android:summary="@string/bigram_prediction_summary" android:summary="@string/bigram_prediction_summary"
android:persistent="true" android:persistent="true"

View file

@ -149,10 +149,13 @@ public class Settings extends InputMethodSettingsFragment
generalSettings.removePreference(mVoicePreference); generalSettings.removePreference(mVoicePreference);
} }
final PreferenceGroup advancedSettings =
(PreferenceGroup) findPreference(PREF_ADVANCED_SETTINGS);
// Remove those meaningless options for now. TODO: delete them for good
advancedSettings.removePreference(findPreference(PREF_BIGRAM_SUGGESTION));
advancedSettings.removePreference(findPreference(PREF_KEY_ENABLE_SPAN_INSERT));
if (!VibratorUtils.getInstance(context).hasVibrator()) { if (!VibratorUtils.getInstance(context).hasVibrator()) {
generalSettings.removePreference(findPreference(PREF_VIBRATE_ON)); generalSettings.removePreference(findPreference(PREF_VIBRATE_ON));
final PreferenceGroup advancedSettings =
(PreferenceGroup) findPreference(PREF_ADVANCED_SETTINGS);
if (null != advancedSettings) { // Theoretically advancedSettings cannot be null if (null != advancedSettings) { // Theoretically advancedSettings cannot be null
advancedSettings.removePreference(findPreference(PREF_VIBRATION_DURATION_SETTINGS)); advancedSettings.removePreference(findPreference(PREF_VIBRATION_DURATION_SETTINGS));
} }

View file

@ -151,8 +151,8 @@ public class SettingsValues {
&& isBigramSuggestionEnabled(prefs, res, mAutoCorrectEnabled); && isBigramSuggestionEnabled(prefs, res, mAutoCorrectEnabled);
mBigramPredictionEnabled = mBigramSuggestionEnabled mBigramPredictionEnabled = mBigramSuggestionEnabled
&& isBigramPredictionEnabled(prefs, res); && isBigramPredictionEnabled(prefs, res);
mEnableSuggestionSpanInsertion = // TODO: remove mEnableSuggestionSpanInsertion. It's always true.
prefs.getBoolean(Settings.PREF_KEY_ENABLE_SPAN_INSERT, true); mEnableSuggestionSpanInsertion = true;
mVibrationDurationSettingsRawValue = mVibrationDurationSettingsRawValue =
prefs.getInt(Settings.PREF_VIBRATION_DURATION_SETTINGS, -1); prefs.getInt(Settings.PREF_VIBRATION_DURATION_SETTINGS, -1);
mKeypressSoundVolumeRawValue = prefs.getFloat(Settings.PREF_KEYPRESS_SOUND_VOLUME, -1.0f); mKeypressSoundVolumeRawValue = prefs.getFloat(Settings.PREF_KEYPRESS_SOUND_VOLUME, -1.0f);
@ -288,13 +288,8 @@ public class SettingsValues {
private static boolean isBigramSuggestionEnabled(final SharedPreferences sp, private static boolean isBigramSuggestionEnabled(final SharedPreferences sp,
final Resources resources, final boolean autoCorrectEnabled) { final Resources resources, final boolean autoCorrectEnabled) {
final boolean showBigramSuggestionsOption = resources.getBoolean( // TODO: remove this method. Bigram suggestion is always true.
R.bool.config_enable_next_word_suggestions_option); return true;
if (!showBigramSuggestionsOption) {
return autoCorrectEnabled;
}
return sp.getBoolean(Settings.PREF_BIGRAM_SUGGESTION, resources.getBoolean(
R.bool.config_default_next_word_suggestions));
} }
private static boolean isBigramPredictionEnabled(final SharedPreferences sp, private static boolean isBigramPredictionEnabled(final SharedPreferences sp,