diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java index 7b7d3822c0b39..344f9cfdbc96d 100644 --- a/services/core/java/com/android/server/pm/PackageManagerService.java +++ b/services/core/java/com/android/server/pm/PackageManagerService.java @@ -13882,7 +13882,7 @@ public class PackageManagerService extends IPackageManager.Stub final boolean isCallerOwner = isCallerDeviceOrProfileOwner(userId); final long callingId = Binder.clearCallingIdentity(); try { - final String activeLauncherPackageName = getActiveLauncherPackageName(userId); + final String activeLauncherPackageName = mPermissionManager.getDefaultHome(userId); final String dialerPackageName = mPermissionManager.getDefaultDialer(userId); for (int i = 0; i < packageNames.length; i++) { canSuspend[i] = false; @@ -13958,18 +13958,6 @@ public class PackageManagerService extends IPackageManager.Stub return canSuspend; } - private String getActiveLauncherPackageName(int userId) { - Intent intent = new Intent(Intent.ACTION_MAIN); - intent.addCategory(Intent.CATEGORY_HOME); - ResolveInfo resolveInfo = resolveIntent( - intent, - intent.resolveTypeIfNeeded(mContext.getContentResolver()), - PackageManager.MATCH_DEFAULT_ONLY, - userId); - - return resolveInfo == null ? null : resolveInfo.activityInfo.packageName; - } - @Override public void verifyPendingInstall(int id, int verificationCode) throws RemoteException { mContext.enforceCallingOrSelfPermission( @@ -20644,6 +20632,9 @@ public class PackageManagerService extends IPackageManager.Stub if (cn != null) { return cn; } + // TODO: This should not happen since there should always be a default package set for + // ROLE_HOME in RoleManager. Continue with a warning log for now. + Slog.w(TAG, "Default package for ROLE_HOME is not set in RoleManager"); // Find the launcher with the highest priority and return that component if there are no // other home activity with the same priority. @@ -20692,6 +20683,7 @@ public class PackageManagerService extends IPackageManager.Stub if (packageName == null) { return null; } + int resolveInfosSize = resolveInfos.size(); for (int i = 0; i < resolveInfosSize; i++) { ResolveInfo resolveInfo = resolveInfos.get(i); @@ -20751,6 +20743,11 @@ public class PackageManagerService extends IPackageManager.Stub // PermissionController manages default home directly. return false; } + + if (packageName == null) { + // Keep the default home package in RoleManager. + return false; + } mPermissionManager.setDefaultHome(packageName, userId, (successful) -> { if (successful) { postPreferredActivityChangedBroadcast(userId);