tree de4947bc81ace9141a44306b0c261f2dfee8b8f9
parent fc1b9998ca8a9fceba47d67fd9ea9b45705b53e0
author Evan Severson <evanseverson@google.com> 1649810201 -0700
committer Android Build Coastguard Worker <android-build-coastguard-worker@google.com> 1683830436 +0000

Watch uid proc state instead of importance for 1-time permissions

The system process may bind to an app with the flag
BIND_FOREGROUND_SERVICE, this will put the client in the foreground
service importance level without the normal requirement that foreground
services must show a notification. Looking at proc states instead allows
us to differentiate between these two levels of foreground service and
revoke the client when not in use.

This change makes the parameters `importanceToResetTimer` and
`importanceToKeepSessionAlive` in PermissionManager#startOneTimePermissionSession
obsolete.

Test: atest CtsPermissionTestCases + manual testing with mic/cam/loc
Bug: 217981062
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:0be78fbbf7d92bf29858aa0c48b171045ab5057f)
Merged-In: I7a725647c001062d1a76a82b680a02e3e2edcb03
Change-Id: I7a725647c001062d1a76a82b680a02e3e2edcb03