PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update permission for accessing Brightness Level

Browse Source 
Bug: 388372939
Flag: com.android.settings.flags.catalyst_display_settings_screen
Test: atest PreferenceServiceTest
NO_IFTTT=Not applicable to preference controller

Change-Id: I7126ca56eb19fd113ad64eed1417651758711f3f
This commit is contained in:
Chris Antol
2025-01-27 20:05:34 +00:00
parent 7e6cc163d9
commit 5015b5c549
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/com/android/settings/display/BrightnessLevelPreference.kt
View File

@@ -97,8 +97,14 @@ class BrightnessLevelPreference :
preference.isPersistent = false
}
override fun getReadPermissions(context: Context) =
Permissions.allOf(Manifest.permission.CONTROL_DISPLAY_BRIGHTNESS)
// Note that we *currently* use restricted APIs to obtain the brightness value, but the
// brightness value itself is not data that requires permission restriction (as evidence this
// value is already available via the summary field). As long as we only
// expose the brightness value and not the other data obtained from BrightnessInfo, we do not
// require a permission control to obtain this value. A new API is in the works for Brightness
// that we can later migrate to that doesn't use any of these permission controls.
// Reference ticket: b/388557367
override fun getReadPermissions(context: Context) = Permissions.EMPTY
override fun getWritePermissions(context: Context) = Permissions.EMPTY