From c83d94015ac5d92117c74b874c29a311c15ff494 Mon Sep 17 00:00:00 2001 From: Felipe Leme Date: Thu, 7 Oct 2021 11:43:23 -0700 Subject: [PATCH] Rename resetNewUserDisclaimer() to acknowledgeNewUserDisclaimer() Test: m update-api Bug: 215375992 Fixes: 202420426 Change-Id: I5670b1cb1206ad0a361bbd91b0a785a6b8075fdf Merged-In: I5670b1cb1206ad0a361bbd91b0a785a6b8075fdf (cherry picked from commit 1e7e2ecee02007859d616e93234cf440a5951717) --- .../android/app/admin/DevicePolicyManager.java | 15 +++++++++++---- .../android/app/admin/IDevicePolicyManager.aidl | 2 +- core/res/AndroidManifest.xml | 2 +- .../devicepolicy/DevicePolicyManagerService.java | 2 +- 4 files changed, 14 insertions(+), 7 deletions(-) diff --git a/core/java/android/app/admin/DevicePolicyManager.java b/core/java/android/app/admin/DevicePolicyManager.java index 3553748e50040..b949ed9d60791 100644 --- a/core/java/android/app/admin/DevicePolicyManager.java +++ b/core/java/android/app/admin/DevicePolicyManager.java @@ -3103,11 +3103,18 @@ public class DevicePolicyManager { } } - /** @hide */ - public void resetNewUserDisclaimer() { + /** + * Acknoledges that the new managed user disclaimer was viewed by the (human) user + * so that {@link #ACTION_SHOW_NEW_USER_DISCLAIMER broadcast} is not sent again the next time + * this user is switched to. + * + * @hide + */ + @RequiresPermission(android.Manifest.permission.MANAGE_USERS) + public void acknowledgeNewUserDisclaimer() { if (mService != null) { try { - mService.resetNewUserDisclaimer(); + mService.acknowledgeNewUserDisclaimer(); } catch (RemoteException e) { throw e.rethrowFromSystemServer(); } @@ -5673,7 +5680,7 @@ public class DevicePolicyManager { */ @SdkConstant(SdkConstantType.BROADCAST_INTENT_ACTION) public static final String ACTION_SHOW_NEW_USER_DISCLAIMER = - "android.app.action.ACTION_SHOW_NEW_USER_DISCLAIMER"; + "android.app.action.SHOW_NEW_USER_DISCLAIMER"; /** * Widgets are enabled in keyguard diff --git a/core/java/android/app/admin/IDevicePolicyManager.aidl b/core/java/android/app/admin/IDevicePolicyManager.aidl index 7c7478bdb41fb..a2863bdfc35dc 100644 --- a/core/java/android/app/admin/IDevicePolicyManager.aidl +++ b/core/java/android/app/admin/IDevicePolicyManager.aidl @@ -264,7 +264,7 @@ interface IDevicePolicyManager { int getLogoutUserId(); void clearLogoutUser(); List getSecondaryUsers(in ComponentName who); - void resetNewUserDisclaimer(); + void acknowledgeNewUserDisclaimer(); void enableSystemApp(in ComponentName admin, in String callerPackage, in String packageName); int enableSystemAppWithIntent(in ComponentName admin, in String callerPackage, in Intent intent); diff --git a/core/res/AndroidManifest.xml b/core/res/AndroidManifest.xml index f525315dd5fa0..2ad836e54b3fb 100644 --- a/core/res/AndroidManifest.xml +++ b/core/res/AndroidManifest.xml @@ -703,7 +703,7 @@ - + diff --git a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java index d681f6985ea52..a46ae27ff8d9a 100644 --- a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java +++ b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java @@ -10763,7 +10763,7 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager { } @Override - public void resetNewUserDisclaimer() { + public void acknowledgeNewUserDisclaimer() { CallerIdentity callerIdentity = getCallerIdentity(); canManageUsers(callerIdentity);