Merge "UserSystemPackageInstaller only (un)installs when appropriate" into rvc-dev am: 9968525afb (e8c7af4d) · Commits · e / os / android_frameworks_base

core/java/android/content/pm/PackageManager.java

+21 −0

Original line number	Diff line number	Diff line
		@@ -1031,6 +1031,27 @@ public abstract class PackageManager {
		*/
		public static final int INSTALL_REASON_ROLLBACK = 5;

		/** @hide */
		@IntDef(prefix = { "UNINSTALL_REASON_" }, value = {
		UNINSTALL_REASON_UNKNOWN,
		UNINSTALL_REASON_USER_TYPE,
		})
		@Retention(RetentionPolicy.SOURCE)
		public @interface UninstallReason {}

		/**
		* Code indicating that the reason for uninstalling this package is unknown.
		* @hide
		*/
		public static final int UNINSTALL_REASON_UNKNOWN = 0;

		/**
		* Code indicating that this package was uninstalled due to the type of user.
		* See UserSystemPackageInstaller
		* @hide
		*/
		public static final int UNINSTALL_REASON_USER_TYPE = 1;

		/**
		* @hide
		*/

core/java/android/content/pm/PackageUserState.java

+7 −0

Original line number	Diff line number	Diff line
		@@ -74,6 +74,7 @@ public class PackageUserState {
		public int appLinkGeneration;
		public int categoryHint = ApplicationInfo.CATEGORY_UNDEFINED;
		public int installReason;
		public @PackageManager.UninstallReason int uninstallReason;
		public String harmfulAppWarning;

		public ArraySet<String> disabledComponents;
		@@ -92,6 +93,7 @@ public class PackageUserState {
		domainVerificationStatus =
		PackageManager.INTENT_FILTER_DOMAIN_VERIFICATION_STATUS_UNDEFINED;
		installReason = PackageManager.INSTALL_REASON_UNKNOWN;
		uninstallReason = PackageManager.UNINSTALL_REASON_UNKNOWN;
		}

		@VisibleForTesting
		@@ -112,6 +114,7 @@ public class PackageUserState {
		appLinkGeneration = o.appLinkGeneration;
		categoryHint = o.categoryHint;
		installReason = o.installReason;
		uninstallReason = o.uninstallReason;
		disabledComponents = ArrayUtils.cloneOrNull(o.disabledComponents);
		enabledComponents = ArrayUtils.cloneOrNull(o.enabledComponents);
		overlayPaths =
		@@ -353,6 +356,9 @@ public class PackageUserState {
		if (installReason != oldState.installReason) {
		return false;
		}
		if (uninstallReason != oldState.uninstallReason) {
		return false;
		}
		if ((disabledComponents == null && oldState.disabledComponents != null)
		\|\| (disabledComponents != null && oldState.disabledComponents == null)) {
		return false;
		@@ -407,6 +413,7 @@ public class PackageUserState {
		hashCode = 31 * hashCode + appLinkGeneration;
		hashCode = 31 * hashCode + categoryHint;
		hashCode = 31 * hashCode + installReason;
		hashCode = 31 * hashCode + uninstallReason;
		hashCode = 31 * hashCode + Objects.hashCode(disabledComponents);
		hashCode = 31 * hashCode + Objects.hashCode(enabledComponents);
		hashCode = 31 * hashCode + Objects.hashCode(harmfulAppWarning);

core/res/res/values/config.xml

+1 −2

Original line number	Diff line number	Diff line
		@@ -2406,8 +2406,7 @@
		- to henceforth disable feature and try to undo its previous effects: 0
		Note: This list must be kept current with PACKAGE_WHITELIST_MODE_PROP in
		frameworks/base/services/core/java/com/android/server/pm/UserSystemPackageInstaller.java -->
		<integer name="config_userTypePackageWhitelistMode">29</integer> <!-- 1+4+8+16 -->
		<!-- TODO(b/143200798): Change to value 13, i.e. 1+4+8, when b/143200798 is resolved. -->
		<integer name="config_userTypePackageWhitelistMode">13</integer> <!-- 1+4+8 -->

		<!-- Whether UI for multi user should be shown -->
		<bool name="config_enableMultiUserUI">false</bool>

data/etc/preinstalled-packages-platform.xml

+2 −2

Original line number	Diff line number	Diff line
		@@ -97,8 +97,8 @@ The way that a device treats system packages that do not have any entry (for any
		is determined by the config resource value config_userTypePackageWhitelistMode.
		See frameworks/base/core/res/res/values/config.xml#config_userTypePackageWhitelistMode.

		Changes to the whitelist during system updates can result in installing new system packages
		to pre-existing users, but cannot uninstall system packages from pre-existing users.
		Changes to the whitelist during system updates can result in installing additional system packages
		to pre-existing users, but cannot uninstall pre-existing system packages from pre-existing users.
		-->
		<config>
		<install-in-user-type package="com.android.providers.settings">

services/core/java/android/content/pm/PackageManagerInternal.java

+0 −17

Original line number	Diff line number	Diff line
		@@ -426,17 +426,6 @@ public abstract class PackageManagerInternal {
		*/
		public abstract String getNameForUid(int uid);

		/**
		* Marks a package as installed (or not installed) for a given user.
		*
		* @param pkg the package whose installation is to be set
		* @param userId the user for whom to set it
		* @param installed the new installed state
		* @return true if the installed state changed as a result
		*/
		public abstract boolean setInstalled(AndroidPackage pkg,
		@UserIdInt int userId, boolean installed);

		/**
		* Request to perform the second phase of ephemeral resolution.
		* @param responseObj The response of the first phase of ephemeral resolution
		@@ -521,12 +510,6 @@ public abstract class PackageManagerInternal {
		*/
		public abstract boolean isPackagePersistent(String packageName);

		/**
		* Returns whether or not the given package represents a legacy system application released
		* prior to runtime permissions.
		*/
		public abstract boolean isLegacySystemApp(AndroidPackage pkg);

		/**
		* Get all overlay packages for a user.
		* @param userId The user for which to get the overlays.