Merge "Expand ASM activation warning." into oc-dr1-dev am: 64acb2906c
am: 4961451a4e
Change-Id: I992843e238acc4720c1fd5ade1cb4a66936ecb9c
This commit is contained in:
@@ -90,9 +90,17 @@ public class AutomaticStorageManagementSwitchPreferenceController extends
|
||||
Settings.Secure.AUTOMATIC_STORAGE_MANAGER_ENABLED,
|
||||
isChecked ? 1 : 0);
|
||||
|
||||
boolean storageManagerEnabledByDefault = SystemProperties.getBoolean(
|
||||
STORAGE_MANAGER_ENABLED_BY_DEFAULT_PROPERTY, false);
|
||||
if (isChecked && !storageManagerEnabledByDefault) {
|
||||
final boolean storageManagerEnabledByDefault =
|
||||
SystemProperties.getBoolean(STORAGE_MANAGER_ENABLED_BY_DEFAULT_PROPERTY, false);
|
||||
final boolean storageManagerDisabledByPolicy =
|
||||
Settings.Secure.getInt(
|
||||
mContext.getContentResolver(),
|
||||
Settings.Secure.AUTOMATIC_STORAGE_MANAGER_TURNED_OFF_BY_POLICY,
|
||||
0)
|
||||
!= 0;
|
||||
// Show warning if it is disabled by default and turning it on or if it was disabled by
|
||||
// policy and we're turning it on.
|
||||
if ((isChecked && (!storageManagerEnabledByDefault || storageManagerDisabledByPolicy))) {
|
||||
ActivationWarningFragment fragment = ActivationWarningFragment.newInstance();
|
||||
fragment.show(mFragmentManager, ActivationWarningFragment.TAG);
|
||||
}
|
||||
|
@@ -168,4 +168,23 @@ public class AutomaticStorageManagementSwitchPreferenceControllerTest {
|
||||
|
||||
verify(transaction, never()).add(any(), eq(ActivationWarningFragment.TAG));
|
||||
}
|
||||
|
||||
@Config(shadows = {SettingsShadowSystemProperties.class})
|
||||
@Test
|
||||
public void togglingOnShouldTriggerWarningFragmentIfEnabledByDefaultAndDisabledByPolicy() {
|
||||
FragmentTransaction transaction = mock(FragmentTransaction.class);
|
||||
when(mFragmentManager.beginTransaction()).thenReturn(transaction);
|
||||
SettingsShadowSystemProperties.set(
|
||||
AutomaticStorageManagementSwitchPreferenceController
|
||||
.STORAGE_MANAGER_ENABLED_BY_DEFAULT_PROPERTY,
|
||||
"true");
|
||||
Settings.Secure.putInt(
|
||||
mContext.getContentResolver(),
|
||||
Settings.Secure.AUTOMATIC_STORAGE_MANAGER_TURNED_OFF_BY_POLICY,
|
||||
1);
|
||||
|
||||
mController.onSwitchToggled(true);
|
||||
|
||||
verify(transaction).add(any(), eq(ActivationWarningFragment.TAG));
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user