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

Commit 5977f60c authored by Tom Natan's avatar Tom Natan Committed by Android (Google) Code Review
Browse files

Merge "Rename PlatformCompat#setOverridesOnReleaseBuilds to...

Merge "Rename PlatformCompat#setOverridesOnReleaseBuilds to putOverridesOnReleaseBuilds." into sc-dev
parents f59889a0 17521947
Loading
Loading
Loading
Loading
+1 −1
Original line number Original line Diff line number Diff line
@@ -119,7 +119,7 @@ public final class CompatChanges {
                ServiceManager.getService(Context.PLATFORM_COMPAT_SERVICE));
                ServiceManager.getService(Context.PLATFORM_COMPAT_SERVICE));
        CompatibilityOverrideConfig config = new CompatibilityOverrideConfig(overrides);
        CompatibilityOverrideConfig config = new CompatibilityOverrideConfig(overrides);
        try {
        try {
            platformCompat.setOverridesOnReleaseBuilds(config, packageName);
            platformCompat.putOverridesOnReleaseBuilds(config, packageName);
        } catch (RemoteException e) {
        } catch (RemoteException e) {
            e.rethrowFromSystemServer();
            e.rethrowFromSystemServer();
        }
        }
+1 −1
Original line number Original line Diff line number Diff line
@@ -168,7 +168,7 @@ interface IPlatformCompat {
     * @param packageName the package name of the app whose changes will be overridden
     * @param packageName the package name of the app whose changes will be overridden
     * @throws SecurityException if overriding changes is not permitted
     * @throws SecurityException if overriding changes is not permitted
     */
     */
    void setOverridesOnReleaseBuilds(in CompatibilityOverrideConfig overrides, in String packageName);
    void putOverridesOnReleaseBuilds(in CompatibilityOverrideConfig overrides, in String packageName);


    /**
    /**
     * Adds overrides to compatibility changes.
     * Adds overrides to compatibility changes.
+4 −4
Original line number Original line Diff line number Diff line
@@ -647,9 +647,9 @@ public final class GameManagerService extends IGameManagerService.Stub {
            final CompatibilityOverrideConfig changeConfig = new CompatibilityOverrideConfig(
            final CompatibilityOverrideConfig changeConfig = new CompatibilityOverrideConfig(
                    overrides);
                    overrides);
            try {
            try {
                mPlatformCompat.setOverridesOnReleaseBuilds(changeConfig, packageName);
                mPlatformCompat.putOverridesOnReleaseBuilds(changeConfig, packageName);
            } catch (RemoteException e) {
            } catch (RemoteException e) {
                Slog.e(TAG, "Failed to call IPlatformCompat#setOverridesOnReleaseBuilds", e);
                Slog.e(TAG, "Failed to call IPlatformCompat#putOverridesOnReleaseBuilds", e);
            }
            }
        } finally {
        } finally {
            Binder.restoreCallingIdentity(uid);
            Binder.restoreCallingIdentity(uid);
@@ -671,9 +671,9 @@ public final class GameManagerService extends IGameManagerService.Stub {
            final CompatibilityOverrideConfig changeConfig = new CompatibilityOverrideConfig(
            final CompatibilityOverrideConfig changeConfig = new CompatibilityOverrideConfig(
                    overrides);
                    overrides);
            try {
            try {
                mPlatformCompat.setOverridesOnReleaseBuilds(changeConfig, packageName);
                mPlatformCompat.putOverridesOnReleaseBuilds(changeConfig, packageName);
            } catch (RemoteException e) {
            } catch (RemoteException e) {
                Slog.e(TAG, "Failed to call IPlatformCompat#setOverridesOnReleaseBuilds", e);
                Slog.e(TAG, "Failed to call IPlatformCompat#putOverridesOnReleaseBuilds", e);
            }
            }
        } finally {
        } finally {
            Binder.restoreCallingIdentity(uid);
            Binder.restoreCallingIdentity(uid);
+9 −9
Original line number Original line Diff line number Diff line
@@ -206,15 +206,6 @@ public class PlatformCompat extends IPlatformCompat.Stub {
        killPackage(packageName);
        killPackage(packageName);
    }
    }


    @Override
    public void setOverridesOnReleaseBuilds(CompatibilityOverrideConfig overrides,
            String packageName) {
        // TODO(b/183630314): Unify the permission enforcement with the other setOverrides* methods.
        checkCompatChangeOverrideOverridablePermission();
        checkAllCompatOverridesAreOverridable(overrides.overrides.keySet());
        mCompatConfig.addOverrides(overrides, packageName);
    }

    @Override
    @Override
    public void setOverridesForTest(CompatibilityChangeConfig overrides, String packageName) {
    public void setOverridesForTest(CompatibilityChangeConfig overrides, String packageName) {
        checkCompatChangeOverridePermission();
        checkCompatChangeOverridePermission();
@@ -229,6 +220,15 @@ public class PlatformCompat extends IPlatformCompat.Stub {
        mCompatConfig.addOverrides(new CompatibilityOverrideConfig(overridesMap), packageName);
        mCompatConfig.addOverrides(new CompatibilityOverrideConfig(overridesMap), packageName);
    }
    }


    @Override
    public void putOverridesOnReleaseBuilds(CompatibilityOverrideConfig overrides,
            String packageName) {
        // TODO(b/183630314): Unify the permission enforcement with the other setOverrides* methods.
        checkCompatChangeOverrideOverridablePermission();
        checkAllCompatOverridesAreOverridable(overrides.overrides.keySet());
        mCompatConfig.addOverrides(overrides, packageName);
    }

    @Override
    @Override
    public int enableTargetSdkChanges(String packageName, int targetSdkVersion) {
    public int enableTargetSdkChanges(String packageName, int targetSdkVersion) {
        checkCompatChangeOverridePermission();
        checkCompatChangeOverridePermission();