diff --git a/core/java/android/provider/Settings.java b/core/java/android/provider/Settings.java index 8b20f0bec7be0..61d8eb13bcf26 100644 --- a/core/java/android/provider/Settings.java +++ b/core/java/android/provider/Settings.java @@ -7880,14 +7880,6 @@ public final class Settings { */ public static final String DEVICE_PAIRED = "device_paired"; - /** - * Integer state indicating whether package verifier is enabled. - * TODO(b/34259924): Remove this setting. - * - * @hide - */ - public static final String PACKAGE_VERIFIER_STATE = "package_verifier_state"; - /** * Specifies additional package name for broadcasting the CMAS messages. * @hide diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java index 365923ea86439..046ffc3c36bb1 100644 --- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java +++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java @@ -2242,9 +2242,6 @@ class SettingsProtoDumpUtil { dumpSetting(s, p, Settings.Secure.PACKAGE_VERIFIER_USER_CONSENT, SecureSettingsProto.PackageVerifier.USER_CONSENT); - dumpSetting(s, p, - Settings.Secure.PACKAGE_VERIFIER_STATE, - SecureSettingsProto.PackageVerifier.STATE); p.end(packageVerifierToken); final long parentalControlToken = p.start(SecureSettingsProto.PARENTAL_CONTROL); diff --git a/packages/SettingsProvider/test/src/android/provider/SettingsBackupTest.java b/packages/SettingsProvider/test/src/android/provider/SettingsBackupTest.java index 6ea3db366f669..3a7de188f9622 100644 --- a/packages/SettingsProvider/test/src/android/provider/SettingsBackupTest.java +++ b/packages/SettingsProvider/test/src/android/provider/SettingsBackupTest.java @@ -17,8 +17,7 @@ package android.provider; import static com.google.android.collect.Sets.newHashSet; - -import static junit.framework.Assert.assertTrue; +import static com.google.common.truth.Truth.assertWithMessage; import static java.lang.reflect.Modifier.isFinal; import static java.lang.reflect.Modifier.isPublic; @@ -655,7 +654,6 @@ public class SettingsBackupTest { Settings.Secure.NIGHT_DISPLAY_LAST_ACTIVATED_TIME, Settings.Secure.NUM_ROTATION_SUGGESTIONS_ACCEPTED, Settings.Secure.ODI_CAPTIONS_ENABLED, - Settings.Secure.PACKAGE_VERIFIER_STATE, Settings.Secure.PACKAGE_VERIFIER_USER_CONSENT, Settings.Secure.PARENTAL_CONTROL_LAST_UPDATE, Settings.Secure.PAYMENT_SERVICE_SEARCH_URI, @@ -754,12 +752,11 @@ public class SettingsBackupTest { Set settings, Set settingsToBackup, Set blacklist) { Set settingsNotBackedUp = difference(settings, settingsToBackup); Set settingsNotBackedUpOrBlacklisted = difference(settingsNotBackedUp, blacklist); - assertTrue( - "Settings not backed up or blacklisted", - settingsNotBackedUpOrBlacklisted.isEmpty()); + assertWithMessage("Settings not backed up or blacklisted") + .that(settingsNotBackedUpOrBlacklisted).isEmpty(); - assertTrue( - "blacklisted settings backed up", intersect(settingsToBackup, blacklist).isEmpty()); + assertWithMessage("blacklisted settings backed up") + .that(intersect(settingsToBackup, blacklist)).isEmpty(); } private static Set getCandidateSettings(