Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8472fdb5 authored by Sam Dubey's avatar Sam Dubey
Browse files

Revert "Update PackageInfo APIs to reflect shared UID status" 

Revert "Update test to check PackageInfo"

Revert submission 17050883-leave_shared_uid_pkginfo

Reason for revert: Breaks AppIntegrityManagerTest* tests
Bug: 223167660
Reverted Changes:
I9a0c5d5ad:Update PackageInfo APIs to reflect shared UID stat...
Ib338b21ab:Update test to check PackageInfo

Change-Id: I7b5debf8c0ba38bf168a4fff36108ceef4e294a8
parent 07ad5ff0

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

+0 −3
Original line number Diff line number Diff line
@@ -121,9 +121,6 @@ public abstract class PackageManager { 
    /** {@hide} */

    public static final boolean APPLY_DEFAULT_TO_DEVICE_PROTECTED_STORAGE = true;



    /** {@hide} */

    public static final boolean ENABLE_SHARED_UID_MIGRATION = true;



    /**

     * This exception is thrown when a given package, application, or component

     * name cannot be found.

core/java/android/content/pm/PackageParser.java

+12 −19
Original line number Diff line number Diff line
@@ -1944,12 +1944,6 @@ public class PackageParser { 
        TypedArray sa = res.obtainAttributes(parser,

                com.android.internal.R.styleable.AndroidManifest);



        int maxSdkVersion = 0;

        if (PackageManager.ENABLE_SHARED_UID_MIGRATION) {

            maxSdkVersion = sa.getInteger(

                    com.android.internal.R.styleable.AndroidManifest_sharedUserMaxSdkVersion, 0);

        }

        if (maxSdkVersion == 0 || maxSdkVersion >= Build.VERSION.RESOURCES_SDK_INT) {

        String str = sa.getNonConfigurationString(

                com.android.internal.R.styleable.AndroidManifest_sharedUserId, 0);

        if (str != null && str.length() > 0) {
@@ -1964,7 +1958,6 @@ public class PackageParser { 
            pkg.mSharedUserLabel = sa.getResourceId(

                    com.android.internal.R.styleable.AndroidManifest_sharedUserLabel, 0);

        }

        }



        pkg.installLocation = sa.getInteger(

                com.android.internal.R.styleable.AndroidManifest_installLocation,

services/core/java/com/android/server/pm/SharedUidMigration.java

+10 −4
Original line number Diff line number Diff line
@@ -17,7 +17,6 @@ 
package com.android.server.pm;



import android.annotation.IntDef;

import android.content.pm.PackageManager;

import android.os.Build;

import android.os.SystemProperties;
@@ -60,15 +59,22 @@ public final class SharedUidMigration { 
    @Retention(RetentionPolicy.SOURCE)

    public @interface Strategy {}



    @Strategy

    private static final int DEFAULT = BEST_EFFORT;



    /**

     * All shared UID migration is disabled.

     * This is not a strategy that can be set with system properties.

     * To disable shared UID migration, change {@link #DEFAULT} to this value.

     */

    private static final int DISABLED = 0;



    /**

     * Whether shared UID migration is fully disabled. Disabled means the sharedUserMaxSdkVersion

     * attribute will be directly ignored in the parsing phase.

     */

    @SuppressWarnings("ConstantConditions")

    public static boolean isDisabled() {

        return !PackageManager.ENABLE_SHARED_UID_MIGRATION;

        return DEFAULT == DISABLED;

    }



    /**
@@ -82,7 +88,7 @@ public final class SharedUidMigration { 


        final int s = SystemProperties.getInt(PROPERTY_KEY, DEFAULT);

        // No transition strategies can be used (http://b/221088088)

        if (s > BEST_EFFORT || s < NEW_INSTALL_ONLY) {

        if (s > BEST_EFFORT || s <= DISABLED) {

            return DEFAULT;

        }

        return s;

services/core/java/com/android/server/pm/parsing/PackageInfoUtils.java

+0 −6
Original line number Diff line number Diff line
@@ -122,12 +122,6 @@ public class PackageInfoUtils { 
        info.isStub = pkg.isStub();

        info.coreApp = pkg.isCoreApp();



        if (!pkgSetting.hasSharedUser()) {

            // It is possible that this shared UID app has left

            info.sharedUserId = null;

            info.sharedUserLabel = 0;

        }



        if ((flags & PackageManager.GET_ACTIVITIES) != 0) {

            final int N = pkg.getActivities().size();

            if (N > 0) {