diff options
7 files changed, 29 insertions, 15 deletions
diff --git a/core/java/android/os/GraphicsEnvironment.java b/core/java/android/os/GraphicsEnvironment.java index d9793097fc3a..b0b8f493e98a 100644 --- a/core/java/android/os/GraphicsEnvironment.java +++ b/core/java/android/os/GraphicsEnvironment.java @@ -480,14 +480,19 @@ public class GraphicsEnvironment { return; } - String applicationPackageName = context.getPackageName(); - String devOptInApplicationName = coreSettings.getString( - Settings.Global.GUP_DEV_OPT_IN_APPS); - boolean devOptIn = applicationPackageName.equals(devOptInApplicationName); - boolean whitelisted = onWhitelist(context, driverPackageName, ai.packageName); - if (!devOptIn && !whitelisted) { + if (getGlobalSettingsString(coreSettings, Settings.Global.GUP_DEV_OPT_OUT_APPS) + .contains(ai.packageName)) { if (DEBUG) { - Log.w(TAG, applicationPackageName + " is not on the whitelist."); + Log.w(TAG, ai.packageName + " opts out from GUP."); + } + return; + } + + if (!getGlobalSettingsString(coreSettings, Settings.Global.GUP_DEV_OPT_IN_APPS) + .contains(ai.packageName) + && !onWhitelist(context, driverPackageName, ai.packageName)) { + if (DEBUG) { + Log.w(TAG, ai.packageName + " is not on the whitelist."); } return; } diff --git a/core/java/android/provider/Settings.java b/core/java/android/provider/Settings.java index 5bb5ee880398..16f9a43de28d 100644 --- a/core/java/android/provider/Settings.java +++ b/core/java/android/provider/Settings.java @@ -12000,12 +12000,18 @@ public final class Settings { "angle_gl_driver_selection_values"; /** - * Apps that are selected to use Game Update Package. + * List of Apps selected to use Game Update Packages. * @hide */ public static final String GUP_DEV_OPT_IN_APPS = "gup_dev_opt_in_apps"; /** + * List of Apps selected not to use Game Update Packages. + * @hide + */ + public static final String GUP_DEV_OPT_OUT_APPS = "gup_dev_opt_out_apps"; + + /** * Apps on the black list that are forbidden to useGame Update Package. * @hide */ diff --git a/core/proto/android/providers/settings/global.proto b/core/proto/android/providers/settings/global.proto index d817aa33302d..a00fde9ed0e3 100644 --- a/core/proto/android/providers/settings/global.proto +++ b/core/proto/android/providers/settings/global.proto @@ -436,11 +436,12 @@ message GlobalSettingsProto { // Ordered GPU debug layer list for GLES // i.e. <layer1>:<layer2>:...:<layerN> optional SettingProto debug_layers_gles = 7; - // Apps opt in to load graphics driver from Game Update Package - // instead of native graphcis driver through developer options. + // GUP - List of Apps selected to use Game Update Packages optional SettingProto gup_dev_opt_in_apps = 8; - // Apps on the black list that are forbidden to useGame Update Package. - optional SettingProto gup_black_list = 9; + // GUP - List of Apps selected not to use Game Update Packages + optional SettingProto gup_dev_opt_out_apps = 9; + // GUP - List of Apps that are forbidden to use Game Update Packages + optional SettingProto gup_black_list = 10; } optional Gpu gpu = 59; diff --git a/core/tests/coretests/src/android/provider/SettingsBackupTest.java b/core/tests/coretests/src/android/provider/SettingsBackupTest.java index 70267c7c2dfa..f8633565cc86 100644 --- a/core/tests/coretests/src/android/provider/SettingsBackupTest.java +++ b/core/tests/coretests/src/android/provider/SettingsBackupTest.java @@ -478,6 +478,7 @@ public class SettingsBackupTest { Settings.Global.GLOBAL_SETTINGS_ANGLE_GL_DRIVER_SELECTION_PKGS, Settings.Global.GLOBAL_SETTINGS_ANGLE_GL_DRIVER_SELECTION_VALUES, Settings.Global.GUP_DEV_OPT_IN_APPS, + Settings.Global.GUP_DEV_OPT_OUT_APPS, Settings.Global.GUP_BLACK_LIST, Settings.Global.GPU_DEBUG_LAYER_APP, Settings.Global.ENABLE_GNSS_RAW_MEAS_FULL_TRACKING, diff --git a/packages/SettingsLib/res/values/strings.xml b/packages/SettingsLib/res/values/strings.xml index 7dcc3acb1dae..03c620580423 100644 --- a/packages/SettingsLib/res/values/strings.xml +++ b/packages/SettingsLib/res/values/strings.xml @@ -1137,9 +1137,6 @@ <!-- The notice header of Third-party licenses. not translatable --> <string name="notice_header" translatable="false"></string> - <!-- UI debug setting: opt in to use updated graphics driver? [CHAR LIMIT=100] --> - <string name="gup_dev_opt_in_app_summary">Opt in app to use Game Update Package in developement</string> - <!-- Name of the phone device [CHAR LIMIT=NONE] --> <string name="media_transfer_phone_device_name">Phone speaker</string> </resources> diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java index 526efcb35262..419273ee28bb 100644 --- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java +++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java @@ -708,6 +708,9 @@ class SettingsProtoDumpUtil { Settings.Global.GUP_DEV_OPT_IN_APPS, GlobalSettingsProto.Gpu.GUP_DEV_OPT_IN_APPS); dumpSetting(s, p, + Settings.Global.GUP_DEV_OPT_OUT_APPS, + GlobalSettingsProto.Gpu.GUP_DEV_OPT_OUT_APPS); + dumpSetting(s, p, Settings.Global.GUP_BLACK_LIST, GlobalSettingsProto.Gpu.GUP_BLACK_LIST); p.end(gpuToken); diff --git a/services/core/java/com/android/server/am/CoreSettingsObserver.java b/services/core/java/com/android/server/am/CoreSettingsObserver.java index f9a77af631af..d7cb2bd36bde 100644 --- a/services/core/java/com/android/server/am/CoreSettingsObserver.java +++ b/services/core/java/com/android/server/am/CoreSettingsObserver.java @@ -68,6 +68,7 @@ final class CoreSettingsObserver extends ContentObserver { sGlobalSettingToTypeMap.put(Settings.Global.GPU_DEBUG_LAYER_APP, String.class); sGlobalSettingToTypeMap.put(Settings.Global.SMS_ACCESS_RESTRICTION_ENABLED, int.class); sGlobalSettingToTypeMap.put(Settings.Global.GUP_DEV_OPT_IN_APPS, String.class); + sGlobalSettingToTypeMap.put(Settings.Global.GUP_DEV_OPT_OUT_APPS, String.class); sGlobalSettingToTypeMap.put(Settings.Global.GUP_BLACK_LIST, String.class); // add other global settings here... } |