Do not review removed permissions
Removed permissions don't participate in the permission model, hence we
should ignore them everywhere.
The problem was that if we set them as "review-required" other code gets
confused as an app that has been reviewed still has some permissions with
this flag set.
Test: 1. Installed legacy app that needs storage access (verified
permission flags)
2. reviewed permisisons (verified permission flags)
3. reset app (verified permission flags)
atest android.appsecurity.cts.PermissionsHostTest#testCompatDefault22 (while in isolated storage mode)
atest android.appsecurity.cts.PermissionsHostTest#testReviewPermissionWhenServiceIsBound (while in isolated storage mode)
Fixes: 120223984
Change-Id: I4b55cc4fb2e39b358fd3b6c5db600c5b3b927096
3 files changed