From 99a66b5416b359abe1b2a94a0a26d7af8e0e9115 Mon Sep 17 00:00:00 2001 From: Phil Weaver Date: Fri, 9 Sep 2016 17:36:03 -0700 Subject: [PATCH] Make a11y settings display consistent We were using two methods to decide whether or not to show that an a11y service was enabled, one on the main Accessibility screen and another on the individual service screen. In some situations, an enabled service would show up as enabled on the main screen but not enabled on its individual screen. Bug: 30829589 Change-Id: I560d86684a2fb8987d3dd06279e033ecb7cc1a58 --- .../ToggleAccessibilityServicePreferenceFragment.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java b/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java index e37de68206d..196a4ab3729 100644 --- a/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java @@ -203,10 +203,8 @@ public class ToggleAccessibilityServicePreferenceFragment } private void updateSwitchBarToggleSwitch() { - final String settingValue = Settings.Secure.getString(getContentResolver(), - Settings.Secure.ENABLED_ACCESSIBILITY_SERVICES); - final boolean checked = settingValue != null - && settingValue.contains(mComponentName.flattenToString()); + final boolean checked = AccessibilityUtils.getEnabledServicesFromSettings(getActivity()) + .contains(mComponentName); mSwitchBar.setCheckedInternal(checked); }