From deb33311db682a584a7981bc02d7c4fb1796738d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E2=80=9CAyush?= Date: Mon, 26 Jul 2021 15:11:13 +0000 Subject: [PATCH] Remove isSeparateProfileChallengeAllowed API This API is not required by framework anymore. Bug: 189502721 Test: N/A Change-Id: Id24d365a0f7bf65af50c147c3655cb5f8c83e1a4 --- .../android/app/admin/DevicePolicyManager.java | 16 ---------------- .../android/app/admin/IDevicePolicyManager.aidl | 2 -- .../internal/widget/LockPatternUtils.java | 8 -------- .../devicepolicy/DevicePolicyManagerService.java | 11 ----------- 4 files changed, 37 deletions(-) diff --git a/core/java/android/app/admin/DevicePolicyManager.java b/core/java/android/app/admin/DevicePolicyManager.java index 27db25de7173f..3cd07ebaeeb79 100644 --- a/core/java/android/app/admin/DevicePolicyManager.java +++ b/core/java/android/app/admin/DevicePolicyManager.java @@ -3269,22 +3269,6 @@ public class DevicePolicyManager { return false; } - /** - * Returns true if the Profile Challenge is available to use for the given profile user. - * - * @hide - */ - public boolean isSeparateProfileChallengeAllowed(int userHandle) { - if (mService != null) { - try { - return mService.isSeparateProfileChallengeAllowed(userHandle); - } catch (RemoteException e) { - throw e.rethrowFromSystemServer(); - } - } - return false; - } - /** * Constant for {@link #setPasswordQuality}: the policy has no requirements * for the password. Note that quality constants are ordered so that higher diff --git a/core/java/android/app/admin/IDevicePolicyManager.aidl b/core/java/android/app/admin/IDevicePolicyManager.aidl index b6c48a1c057b5..8cf1f809af92f 100644 --- a/core/java/android/app/admin/IDevicePolicyManager.aidl +++ b/core/java/android/app/admin/IDevicePolicyManager.aidl @@ -373,8 +373,6 @@ interface IDevicePolicyManager { CharSequence getShortSupportMessageForUser(in ComponentName admin, int userHandle); CharSequence getLongSupportMessageForUser(in ComponentName admin, int userHandle); - boolean isSeparateProfileChallengeAllowed(int userHandle); - void setOrganizationColor(in ComponentName admin, in int color); void setOrganizationColorForUser(in int color, in int userId); int getOrganizationColor(in ComponentName admin); diff --git a/core/java/com/android/internal/widget/LockPatternUtils.java b/core/java/com/android/internal/widget/LockPatternUtils.java index 498505cd46fff..dbf45287d9d80 100644 --- a/core/java/com/android/internal/widget/LockPatternUtils.java +++ b/core/java/com/android/internal/widget/LockPatternUtils.java @@ -846,14 +846,6 @@ public class LockPatternUtils { return isManagedProfile(userHandle) && !hasSeparateChallenge(userHandle); } - /** - * Retrieves whether the current DPM allows use of the Profile Challenge. - */ - public boolean isSeparateProfileChallengeAllowed(int userHandle) { - return isManagedProfile(userHandle) - && getDevicePolicyManager().isSeparateProfileChallengeAllowed(userHandle); - } - private boolean hasSeparateChallenge(int userHandle) { try { return getLockSettings().getSeparateProfileChallengeEnabled(userHandle); diff --git a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java index be8b21c3808ff..c6f99c3156b76 100644 --- a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java +++ b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java @@ -3813,17 +3813,6 @@ public class DevicePolicyManagerService extends BaseIDevicePolicyManager { } } - @Override - public boolean isSeparateProfileChallengeAllowed(int userHandle) { - Preconditions.checkCallAuthorization(isSystemUid(getCallerIdentity()), - String.format(NOT_SYSTEM_CALLER_MSG, "query separate challenge support")); - - ComponentName profileOwner = getProfileOwnerAsUser(userHandle); - // Profile challenge is supported on N or newer release. - return profileOwner != null && - getTargetSdk(profileOwner.getPackageName(), userHandle) > Build.VERSION_CODES.M; - } - private boolean canSetPasswordQualityOnParent(String packageName, final CallerIdentity caller) { return !mInjector.isChangeEnabled( PREVENT_SETTING_PASSWORD_QUALITY_ON_PARENT, packageName, caller.getUserId())