diff options
author | Sergey Nikolaienkov <sergeynv@google.com> | 2020-10-08 08:26:14 +0000 |
---|---|---|
committer | Sergey Nikolaienkov <sergeynv@google.com> | 2021-01-20 06:21:07 +0000 |
commit | 9beb0c9299e293ad2a54ce5cbcd26f50cfb9e01d (patch) | |
tree | 25963338d844c8fda8ceedd7495ae00da5ee63af /PermissionController/src/com/android/permissioncontroller/Constants.java | |
parent | d0558e12a7f6d5fafb2bb11fc72c71262e1d0d1b (diff) | |
download | Permission-9beb0c9299e293ad2a54ce5cbcd26f50cfb9e01d.tar.gz |
"Relevant" PermissionController on TV
This change revises UX for mananging permissions on TV devices.
At the core of the change is idea of presenting users with the most
relevant information on the main screen (the first displayed when the
PermissionController application is launched) and moving other
inormation to the "other" permissions screen.
The following is the set of the new UX rules imlemented by this CL.
1. Main screen should only show platform permissions that either have
been requested by non-system applications or have been pre-granted to
non-system applications.
2. Platform permissions used only by system applications and
non-platform (aka "custom") permissions should be moved to a separate
screen, linked from the "main" screen.
3. Permissions not used by any application (either system or non-system)
should not be displayed.
4. Applications that only declare use of a permission, but have never
actually requested it at runtime and do not have the permission
pre-granted to them, should not be counted as "requesting" (one of the Y
in "Y of X allowed").
5. Do not show "Y of X allowed" labels for the system permission on the
"other" permission screen (with the new couting rules all of them would
be "0 of 0 allowed").
In the implementation we introduce ManagePermissionsViewModel
that both ManagePermissionsFragment and ManagerOtherPermissionsFragment
rely on for data loading, maintaining and monitoring.
The ManagePermissionsViewModel is using the new data model isntead of
the "legacy" one, that has been used up until this point.
Bug: 157497448
Bug: 160723997
Test: manually on adt3
Change-Id: I6c6b92871f8dd3caabe6f336ced7de39538b4621
Diffstat (limited to 'PermissionController/src/com/android/permissioncontroller/Constants.java')
-rw-r--r-- | PermissionController/src/com/android/permissioncontroller/Constants.java | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/PermissionController/src/com/android/permissioncontroller/Constants.java b/PermissionController/src/com/android/permissioncontroller/Constants.java index a669c79d5..459b6d35e 100644 --- a/PermissionController/src/com/android/permissioncontroller/Constants.java +++ b/PermissionController/src/com/android/permissioncontroller/Constants.java @@ -195,4 +195,9 @@ public class Constants { * {@link com.android.permissioncontroller.permission.ui.AutoGrantPermissionsNotifier}. */ public static final int PERMISSION_GRANTED_BY_ADMIN_NOTIFICATION_ID = 1; + + /** + * Package name of the Android platform. + */ + public static final String OS_PACKAGE_NAME = "android"; } |