Remove manual pregrants for ContentCaptureService (now handled via roles)

DefaultPermissionGrantPolicy was granting several permissions to the
package that provides ContentCaptureService. Instead, these permissions
are now managed by roles (PermissionController/res/xml/roles.xml).

As a result of this change, the "Android System Intelligence" package
(com.google.android.as) will no longer have these permissions:
```
ACCESS_MEDIA_LOCATION
ADD_VOICEMAIL
CALL_PHONE
GET_ACCOUNTS
PROCESS_OUTGOING_CALLS
READ_CELL_BROADCASTS
RECEIVE_MMS
RECEIVE_SMS
RECEIVE_WAP_PUSH
SEND_SMS
USE_SIP
WRITE_CALL_LOG
WRITE_CONTACTS
WRITE_EXTERNAL_STORAGE
```

"Android System Intelligence" will maintain these permissions (but now
via roles):
```
ACCESS_BACKGROUND_LOCATION
ACCESS_COARSE_LOCATION
ACCESS_FINE_LOCATION
READ_CALL_LOG
READ_CONTACTS
READ_PHONE_STATE
READ_SMS
```

Bug: 222231605
Test: adb shell dumpsys package com.google.android.as
Test: atest GtsPermissionTestCases:com.google.android.permission.gts.DefaultPermissionGrantPolicyTest#testDefaultGrantsWithRemoteExceptions
Change-Id: If78ad9742c14e10e6a6ab04c600881904b0bad51
This commit is contained in:
Nikita Dubrovsky
2022-04-14 16:50:03 -07:00
parent 73a728edfd
commit bcbe73414f

View File

@@ -896,15 +896,6 @@ final class DefaultPermissionGrantPolicy {
COARSE_BACKGROUND_LOCATION_PERMISSIONS, CONTACTS_PERMISSIONS);
}
// Content capture
String contentCapturePackageName =
mContext.getPackageManager().getContentCaptureServicePackageName();
if (!TextUtils.isEmpty(contentCapturePackageName)) {
grantPermissionsToSystemPackage(pm, contentCapturePackageName, userId,
PHONE_PERMISSIONS, SMS_PERMISSIONS, ALWAYS_LOCATION_PERMISSIONS,
CONTACTS_PERMISSIONS, STORAGE_PERMISSIONS);
}
// Attention Service
String attentionServicePackageName =
mContext.getPackageManager().getAttentionServicePackageName();