Skip to content

Commit b09b84c

Browse files
Makoto OnukiAndroid (Google) Code Review
authored andcommitted
Merge "DefaultPermission: GET_ACCOUNTS is now in CONTACTS" into mnc-dev
2 parents db8fa48 + bc92479 commit b09b84c

1 file changed

Lines changed: 2 additions & 10 deletions

File tree

services/core/java/com/android/server/pm/DefaultPermissionGrantPolicy.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ final class DefaultPermissionGrantPolicy {
7272
static {
7373
CONTACTS_PERMISSIONS.add(Manifest.permission.READ_CONTACTS);
7474
CONTACTS_PERMISSIONS.add(Manifest.permission.WRITE_CONTACTS);
75+
CONTACTS_PERMISSIONS.add(Manifest.permission.GET_ACCOUNTS);
7576
}
7677

7778
private static final Set<String> LOCATION_PERMISSIONS = new ArraySet<>();
@@ -117,11 +118,6 @@ final class DefaultPermissionGrantPolicy {
117118
STORAGE_PERMISSIONS.add(Manifest.permission.WRITE_EXTERNAL_STORAGE);
118119
}
119120

120-
private static final Set<String> ACCOUNTS_PERMISSIONS = new ArraySet<>();
121-
static {
122-
ACCOUNTS_PERMISSIONS.add(Manifest.permission.GET_ACCOUNTS);
123-
}
124-
125121
private final PackageManagerService mService;
126122

127123
private PackagesProvider mImePackagesProvider;
@@ -352,7 +348,6 @@ && doesPackageSupportRuntimePermissions(downloadsUiPackage)) {
352348
&& doesPackageSupportRuntimePermissions(calendarPackage)) {
353349
grantRuntimePermissionsLPw(calendarPackage, CALENDAR_PERMISSIONS, userId);
354350
grantRuntimePermissionsLPw(calendarPackage, CONTACTS_PERMISSIONS, userId);
355-
grantRuntimePermissionsLPw(calendarPackage, ACCOUNTS_PERMISSIONS, userId);
356351
}
357352

358353
// Calendar provider
@@ -362,7 +357,6 @@ && doesPackageSupportRuntimePermissions(calendarPackage)) {
362357
grantRuntimePermissionsLPw(calendarProviderPackage, CONTACTS_PERMISSIONS, userId);
363358
grantRuntimePermissionsLPw(calendarProviderPackage, CALENDAR_PERMISSIONS,
364359
true, userId);
365-
grantRuntimePermissionsLPw(calendarProviderPackage, ACCOUNTS_PERMISSIONS, userId);
366360
grantRuntimePermissionsLPw(calendarProviderPackage, STORAGE_PERMISSIONS, userId);
367361
}
368362

@@ -386,7 +380,6 @@ && doesPackageSupportRuntimePermissions(calendarPackage)) {
386380
&& doesPackageSupportRuntimePermissions(contactsPackage)) {
387381
grantRuntimePermissionsLPw(contactsPackage, CONTACTS_PERMISSIONS, userId);
388382
grantRuntimePermissionsLPw(contactsPackage, PHONE_PERMISSIONS, userId);
389-
grantRuntimePermissionsLPw(contactsPackage, ACCOUNTS_PERMISSIONS, userId);
390383
}
391384

392385
// Contacts provider sync adapters
@@ -408,7 +401,6 @@ && doesPackageSupportRuntimePermissions(contactsPackage)) {
408401
true, userId);
409402
grantRuntimePermissionsLPw(contactsProviderPackage, PHONE_PERMISSIONS,
410403
true, userId);
411-
grantRuntimePermissionsLPw(contactsProviderPackage, ACCOUNTS_PERMISSIONS, userId);
412404
grantRuntimePermissionsLPw(contactsProviderPackage, STORAGE_PERMISSIONS, userId);
413405
}
414406

@@ -419,7 +411,7 @@ && doesPackageSupportRuntimePermissions(contactsPackage)) {
419411
getDefaultSystemHandlerActivityPackageLPr(deviceProvisionIntent, userId);
420412
if (deviceProvisionPackage != null
421413
&& doesPackageSupportRuntimePermissions(deviceProvisionPackage)) {
422-
grantRuntimePermissionsLPw(deviceProvisionPackage, ACCOUNTS_PERMISSIONS, userId);
414+
grantRuntimePermissionsLPw(deviceProvisionPackage, CONTACTS_PERMISSIONS, userId);
423415
}
424416

425417
// Maps

0 commit comments

Comments
 (0)