Merge "[Compat UI] Enable SCM Restart Dialog by Default" into tm-qpr-dev am: 4e02695e3e
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21073192 Change-Id: I06250a3b4cd4ef3ab7ce4f77c9fcee266c390d84 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -36,11 +36,15 @@ import javax.inject.Inject;
|
|||||||
public class CompatUIConfiguration implements DeviceConfig.OnPropertiesChangedListener {
|
public class CompatUIConfiguration implements DeviceConfig.OnPropertiesChangedListener {
|
||||||
|
|
||||||
private static final String KEY_ENABLE_LETTERBOX_RESTART_DIALOG =
|
private static final String KEY_ENABLE_LETTERBOX_RESTART_DIALOG =
|
||||||
"enable_letterbox_restart_dialog";
|
"enable_letterbox_restart_confirmation_dialog";
|
||||||
|
|
||||||
private static final String KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION =
|
private static final String KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION =
|
||||||
"enable_letterbox_reachability_education";
|
"enable_letterbox_reachability_education";
|
||||||
|
|
||||||
|
private static final boolean DEFAULT_VALUE_ENABLE_LETTERBOX_RESTART_DIALOG = true;
|
||||||
|
|
||||||
|
private static final boolean DEFAULT_VALUE_ENABLE_LETTERBOX_REACHABILITY_EDUCATION = false;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* The name of the {@link SharedPreferences} that holds which user has seen the Restart
|
* The name of the {@link SharedPreferences} that holds which user has seen the Restart
|
||||||
* confirmation dialog.
|
* confirmation dialog.
|
||||||
@@ -77,10 +81,11 @@ public class CompatUIConfiguration implements DeviceConfig.OnPropertiesChangedLi
|
|||||||
mIsReachabilityEducationEnabled = context.getResources().getBoolean(
|
mIsReachabilityEducationEnabled = context.getResources().getBoolean(
|
||||||
R.bool.config_letterboxIsReachabilityEducationEnabled);
|
R.bool.config_letterboxIsReachabilityEducationEnabled);
|
||||||
mIsLetterboxRestartDialogAllowed = DeviceConfig.getBoolean(
|
mIsLetterboxRestartDialogAllowed = DeviceConfig.getBoolean(
|
||||||
DeviceConfig.NAMESPACE_WINDOW_MANAGER, KEY_ENABLE_LETTERBOX_RESTART_DIALOG, false);
|
DeviceConfig.NAMESPACE_WINDOW_MANAGER, KEY_ENABLE_LETTERBOX_RESTART_DIALOG,
|
||||||
|
DEFAULT_VALUE_ENABLE_LETTERBOX_RESTART_DIALOG);
|
||||||
mIsLetterboxReachabilityEducationAllowed = DeviceConfig.getBoolean(
|
mIsLetterboxReachabilityEducationAllowed = DeviceConfig.getBoolean(
|
||||||
DeviceConfig.NAMESPACE_WINDOW_MANAGER, KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION,
|
DeviceConfig.NAMESPACE_WINDOW_MANAGER, KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION,
|
||||||
false);
|
DEFAULT_VALUE_ENABLE_LETTERBOX_REACHABILITY_EDUCATION);
|
||||||
DeviceConfig.addOnPropertiesChangedListener(DeviceConfig.NAMESPACE_APP_COMPAT, mainExecutor,
|
DeviceConfig.addOnPropertiesChangedListener(DeviceConfig.NAMESPACE_APP_COMPAT, mainExecutor,
|
||||||
this);
|
this);
|
||||||
mSharedPreferences = context.getSharedPreferences(DONT_SHOW_RESTART_DIALOG_PREF_NAME,
|
mSharedPreferences = context.getSharedPreferences(DONT_SHOW_RESTART_DIALOG_PREF_NAME,
|
||||||
@@ -133,16 +138,17 @@ public class CompatUIConfiguration implements DeviceConfig.OnPropertiesChangedLi
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onPropertiesChanged(@NonNull DeviceConfig.Properties properties) {
|
public void onPropertiesChanged(@NonNull DeviceConfig.Properties properties) {
|
||||||
// TODO(b/263349751): Update flag and default value to true
|
|
||||||
if (properties.getKeyset().contains(KEY_ENABLE_LETTERBOX_RESTART_DIALOG)) {
|
if (properties.getKeyset().contains(KEY_ENABLE_LETTERBOX_RESTART_DIALOG)) {
|
||||||
mIsLetterboxRestartDialogAllowed = DeviceConfig.getBoolean(
|
mIsLetterboxRestartDialogAllowed = DeviceConfig.getBoolean(
|
||||||
DeviceConfig.NAMESPACE_WINDOW_MANAGER, KEY_ENABLE_LETTERBOX_RESTART_DIALOG,
|
DeviceConfig.NAMESPACE_WINDOW_MANAGER, KEY_ENABLE_LETTERBOX_RESTART_DIALOG,
|
||||||
false);
|
DEFAULT_VALUE_ENABLE_LETTERBOX_RESTART_DIALOG);
|
||||||
}
|
}
|
||||||
|
// TODO(b/263349751): Update flag and default value to true
|
||||||
if (properties.getKeyset().contains(KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION)) {
|
if (properties.getKeyset().contains(KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION)) {
|
||||||
mIsLetterboxReachabilityEducationAllowed = DeviceConfig.getBoolean(
|
mIsLetterboxReachabilityEducationAllowed = DeviceConfig.getBoolean(
|
||||||
DeviceConfig.NAMESPACE_WINDOW_MANAGER,
|
DeviceConfig.NAMESPACE_WINDOW_MANAGER,
|
||||||
KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION, false);
|
KEY_ENABLE_LETTERBOX_REACHABILITY_EDUCATION,
|
||||||
|
DEFAULT_VALUE_ENABLE_LETTERBOX_REACHABILITY_EDUCATION);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user