Skip to content
Snippets Groups Projects
Commit eebbe541 authored by Ken Wakasa's avatar Ken Wakasa Committed by Android (Google) Code Review
Browse files

Merge "Fix password variation check code"

parents 80275c7d 16ee377b
No related branches found
No related tags found
No related merge requests found
...@@ -494,10 +494,24 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen ...@@ -494,10 +494,24 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
return container; return container;
} }
private static boolean isPasswordVariation(int variation) { // Please refer to TextView.isPasswordInputType
return variation == InputType.TYPE_TEXT_VARIATION_PASSWORD private static boolean isPasswordInputType(int inputType) {
|| variation == InputType.TYPE_TEXT_VARIATION_VISIBLE_PASSWORD final int variation =
|| variation == InputType.TYPE_TEXT_VARIATION_WEB_PASSWORD; inputType & (InputType.TYPE_MASK_CLASS | InputType.TYPE_MASK_VARIATION);
return (variation
== (InputType.TYPE_CLASS_TEXT | InputType.TYPE_TEXT_VARIATION_PASSWORD))
|| (variation
== (InputType.TYPE_CLASS_TEXT | InputType.TYPE_TEXT_VARIATION_WEB_PASSWORD))
|| (variation
== (InputType.TYPE_CLASS_NUMBER | InputType.TYPE_NUMBER_VARIATION_PASSWORD));
}
// Please refer to TextView.isVisiblePasswordInputType
private static boolean isVisiblePasswordInputType(int inputType) {
final int variation =
inputType & (InputType.TYPE_MASK_CLASS | InputType.TYPE_MASK_VARIATION);
return variation
== (InputType.TYPE_CLASS_TEXT | InputType.TYPE_TEXT_VARIATION_VISIBLE_PASSWORD);
} }
private static boolean isEmailVariation(int variation) { private static boolean isEmailVariation(int variation) {
...@@ -525,8 +539,8 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen ...@@ -525,8 +539,8 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
// Most such things we decide below in initializeInputAttributesAndGetMode, but we need to // Most such things we decide below in initializeInputAttributesAndGetMode, but we need to
// know now whether this is a password text field, because we need to know now whether we // know now whether this is a password text field, because we need to know now whether we
// want to enable the voice button. // want to enable the voice button.
mVoiceConnector.resetVoiceStates(isPasswordVariation( mVoiceConnector.resetVoiceStates(isPasswordInputType(attribute.inputType)
attribute.inputType & InputType.TYPE_MASK_VARIATION)); || isVisiblePasswordInputType(attribute.inputType));
final int mode = initializeInputAttributesAndGetMode(attribute.inputType); final int mode = initializeInputAttributesAndGetMode(attribute.inputType);
...@@ -583,7 +597,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen ...@@ -583,7 +597,7 @@ public class LatinIME extends InputMethodService implements KeyboardActionListen
case InputType.TYPE_CLASS_TEXT: case InputType.TYPE_CLASS_TEXT:
mIsSettingsSuggestionStripOn = true; mIsSettingsSuggestionStripOn = true;
// Make sure that passwords are not displayed in candidate view // Make sure that passwords are not displayed in candidate view
if (isPasswordVariation(variation)) { if (isPasswordInputType(inputType) || isVisiblePasswordInputType(inputType)) {
mIsSettingsSuggestionStripOn = false; mIsSettingsSuggestionStripOn = false;
} }
if (isEmailVariation(variation) if (isEmailVariation(variation)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment