Merge "Enable notification permission feature"

This commit is contained in:
TreeHugger Robot
2022-02-05 15:01:29 +00:00
committed by Android (Google) Code Review
2 changed files with 11 additions and 4 deletions

View File

@@ -3624,7 +3624,7 @@ public class SettingsProvider extends ContentProvider {
}
private final class UpgradeController {
private static final int SETTINGS_VERSION = 209;
private static final int SETTINGS_VERSION = 210;
private final int mUserId;
@@ -5498,17 +5498,21 @@ public class SettingsProvider extends ContentProvider {
}
if (currentVersion == 208) {
// Version 208: Enable enforcement of
// Unused
currentVersion = 209;
}
if (currentVersion == 209) {
// Version 209: Enable enforcement of
// android.Manifest.permission#POST_NOTIFICATIONS in order for applications
// to post notifications.
final SettingsState secureSettings = getSecureSettingsLocked(userId);
secureSettings.insertSettingLocked(
Secure.NOTIFICATION_PERMISSION_ENABLED,
/* enabled= */" 1",
/* enabled= */ "1",
/* tag= */ null,
/* makeDefault= */ false,
SettingsState.SYSTEM_PACKAGE_NAME);
currentVersion = 209;
currentVersion = 210;
}
// vXXX: Add new settings above this point.

View File

@@ -372,6 +372,9 @@ public class NotificationManagerServiceTest extends UiServiceTestCase {
"android.permission.WRITE_DEVICE_CONFIG",
"android.permission.READ_DEVICE_CONFIG",
"android.permission.READ_CONTACTS");
Settings.Secure.putIntForUser(
getContext().getContentResolver(),
Settings.Secure.NOTIFICATION_PERMISSION_ENABLED, 0, USER_SYSTEM);
MockitoAnnotations.initMocks(this);