Merge "Fix the subtype id of the additional subtypes"

main
Satoshi Kataoka 2013-10-03 07:45:24 +00:00 committed by Android (Google) Code Review
commit 85607c5965
1 changed files with 25 additions and 4 deletions

View File

@ -61,10 +61,8 @@ public final class AdditionalSubtypeUtils {
StringUtils.appendToCommaSplittableTextIfNotExists(
IS_ADDITIONAL_SUBTYPE, layoutDisplayNameExtraValue);
final int nameId = SubtypeLocaleUtils.getSubtypeNameId(localeString, keyboardLayoutSetName);
return new InputMethodSubtype(nameId, R.drawable.ic_ime_switcher_dark,
localeString, KEYBOARD_MODE, layoutExtraValue + "," + additionalSubtypeExtraValue
+ "," + Constants.Subtype.ExtraValue.ASCII_CAPABLE
+ "," + Constants.Subtype.ExtraValue.EMOJI_CAPABLE, false, false);
return buildInputMethodSubtype(
nameId, localeString, layoutExtraValue, additionalSubtypeExtraValue);
}
public static String getPrefSubtype(final InputMethodSubtype subtype) {
@ -137,4 +135,27 @@ public final class AdditionalSubtypeUtils {
}
return sb.toString();
}
private static InputMethodSubtype buildInputMethodSubtype(int nameId, String localeString,
String layoutExtraValue, String additionalSubtypeExtraValue) {
// CAVEAT! If you want to change subtypeId after changing the extra values,
// you must change "getInputMethodSubtypeId". But it will remove the additional keyboard
// from the current users. So, you should be really careful to change it.
final int subtypeId = getInputMethodSubtypeId(nameId, localeString, layoutExtraValue,
additionalSubtypeExtraValue);
// TODO: Use InputMethodSubtypeBuilder once we use SDK version 19.
return new InputMethodSubtype(nameId, R.drawable.ic_ime_switcher_dark,
localeString, KEYBOARD_MODE, layoutExtraValue + "," + additionalSubtypeExtraValue
+ "," + Constants.Subtype.ExtraValue.ASCII_CAPABLE
+ "," + Constants.Subtype.ExtraValue.EMOJI_CAPABLE, false, false,
subtypeId);
}
private static int getInputMethodSubtypeId(int nameId, String localeString,
String layoutExtraValue, String additionalSubtypeExtraValue) {
// TODO: Use InputMethodSubtypeBuilder once we use SDK version 19.
return (new InputMethodSubtype(nameId, R.drawable.ic_ime_switcher_dark,
localeString, KEYBOARD_MODE, layoutExtraValue + "," + additionalSubtypeExtraValue,
false, false)).hashCode();
}
}