Merge "Show the accessibility usage preference when Permissions Hub is disabled." into qt-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
51c92e5622
@@ -18,14 +18,12 @@ package com.android.settings.privacy;
|
|||||||
|
|
||||||
import android.accessibilityservice.AccessibilityServiceInfo;
|
import android.accessibilityservice.AccessibilityServiceInfo;
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.provider.DeviceConfig;
|
|
||||||
import android.view.accessibility.AccessibilityManager;
|
import android.view.accessibility.AccessibilityManager;
|
||||||
|
|
||||||
import androidx.annotation.NonNull;
|
import androidx.annotation.NonNull;
|
||||||
import androidx.preference.Preference;
|
import androidx.preference.Preference;
|
||||||
|
|
||||||
import com.android.settings.R;
|
import com.android.settings.R;
|
||||||
import com.android.settings.Utils;
|
|
||||||
import com.android.settings.core.BasePreferenceController;
|
import com.android.settings.core.BasePreferenceController;
|
||||||
|
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
@@ -46,10 +44,7 @@ public class AccessibilityUsagePreferenceController extends BasePreferenceContro
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public int getAvailabilityStatus() {
|
public int getAvailabilityStatus() {
|
||||||
return (mEnabledServiceInfos.isEmpty() || !Boolean.parseBoolean(
|
return mEnabledServiceInfos.isEmpty() ? UNSUPPORTED_ON_DEVICE : AVAILABLE;
|
||||||
DeviceConfig.getProperty(DeviceConfig.NAMESPACE_PRIVACY,
|
|
||||||
Utils.PROPERTY_PERMISSIONS_HUB_ENABLED)))
|
|
||||||
? UNSUPPORTED_ON_DEVICE : AVAILABLE;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
|
@@ -63,16 +63,6 @@ public class AccessibilityUsagePreferenceControllerTest {
|
|||||||
ShadowDeviceConfig.reset();
|
ShadowDeviceConfig.reset();
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
|
||||||
public void isAvailable_permissionHubNotSet_shouldReturnUnsupported() {
|
|
||||||
mAccessibilityManager.setEnabledAccessibilityServiceList(new ArrayList<>());
|
|
||||||
AccessibilityUsagePreferenceController controller =
|
|
||||||
new AccessibilityUsagePreferenceController(mContext, "test_key");
|
|
||||||
|
|
||||||
// We have not yet set the property to show the Permissions Hub.
|
|
||||||
assertThat(controller.getAvailabilityStatus()).isEqualTo(UNSUPPORTED_ON_DEVICE);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getAvailabilityStatus_noEnabledServices_shouldReturnUnsupported() {
|
public void getAvailabilityStatus_noEnabledServices_shouldReturnUnsupported() {
|
||||||
DeviceConfig.setProperty(DeviceConfig.NAMESPACE_PRIVACY,
|
DeviceConfig.setProperty(DeviceConfig.NAMESPACE_PRIVACY,
|
||||||
@@ -85,9 +75,9 @@ public class AccessibilityUsagePreferenceControllerTest {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getAvailabilityStatus_enabledServices_shouldReturnAvailableUnsearchable() {
|
public void getAvailabilityStatus_enabledServices_shouldReturnAvailable() {
|
||||||
DeviceConfig.setProperty(DeviceConfig.NAMESPACE_PRIVACY,
|
DeviceConfig.setProperty(DeviceConfig.NAMESPACE_PRIVACY,
|
||||||
Utils.PROPERTY_PERMISSIONS_HUB_ENABLED, "true", true);
|
Utils.PROPERTY_PERMISSIONS_HUB_ENABLED, "false", true);
|
||||||
mAccessibilityManager.setEnabledAccessibilityServiceList(
|
mAccessibilityManager.setEnabledAccessibilityServiceList(
|
||||||
new ArrayList<>(Arrays.asList(new AccessibilityServiceInfo())));
|
new ArrayList<>(Arrays.asList(new AccessibilityServiceInfo())));
|
||||||
AccessibilityUsagePreferenceController controller =
|
AccessibilityUsagePreferenceController controller =
|
||||||
|
Reference in New Issue
Block a user