summaryrefslogtreecommitdiff
path: root/core/java/android
diff options
context:
space:
mode:
Diffstat (limited to 'core/java/android')
-rw-r--r--core/java/android/app/compat/ChangeIdStateCache.java86
-rw-r--r--core/java/android/app/compat/ChangeIdStateQuery.java87
-rw-r--r--core/java/android/app/compat/CompatChanges.java32
3 files changed, 27 insertions, 178 deletions
diff --git a/core/java/android/app/compat/ChangeIdStateCache.java b/core/java/android/app/compat/ChangeIdStateCache.java
deleted file mode 100644
index 9ef63f6587f0..000000000000
--- a/core/java/android/app/compat/ChangeIdStateCache.java
+++ /dev/null
@@ -1,86 +0,0 @@
-/*
- * Copyright (C) 2020 The Android Open Source Project
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package android.app.compat;
-
-import android.app.PropertyInvalidatedCache;
-import android.content.Context;
-import android.os.Binder;
-import android.os.RemoteException;
-import android.os.ServiceManager;
-
-import com.android.internal.compat.IPlatformCompat;
-
-/**
- * Handles caching of calls to {@link com.android.internal.compat.IPlatformCompat}
- * @hide
- */
-public final class ChangeIdStateCache
- extends PropertyInvalidatedCache<ChangeIdStateQuery, Boolean> {
- private static final String CACHE_KEY = "cache_key.is_compat_change_enabled";
- private static final int MAX_ENTRIES = 20;
- private static boolean sDisabled = false;
-
- /** @hide */
- public ChangeIdStateCache() {
- super(MAX_ENTRIES, CACHE_KEY);
- }
-
- /**
- * Disable cache.
- *
- * <p>Should only be used in unit tests.
- * @hide
- */
- public static void disable() {
- sDisabled = true;
- }
-
- /**
- * Invalidate the cache.
- *
- * <p>Can only be called by the system server process.
- * @hide
- */
- public static void invalidate() {
- if (!sDisabled) {
- PropertyInvalidatedCache.invalidateCache(CACHE_KEY);
- }
- }
-
- @Override
- protected Boolean recompute(ChangeIdStateQuery query) {
- IPlatformCompat platformCompat = IPlatformCompat.Stub.asInterface(
- ServiceManager.getService(Context.PLATFORM_COMPAT_SERVICE));
- final long token = Binder.clearCallingIdentity();
- try {
- if (query.type == ChangeIdStateQuery.QUERY_BY_PACKAGE_NAME) {
- return platformCompat.isChangeEnabledByPackageName(query.changeId,
- query.packageName,
- query.userId);
- } else if (query.type == ChangeIdStateQuery.QUERY_BY_UID) {
- return platformCompat.isChangeEnabledByUid(query.changeId, query.uid);
- } else {
- throw new IllegalArgumentException("Invalid query type: " + query.type);
- }
- } catch (RemoteException e) {
- e.rethrowFromSystemServer();
- } finally {
- Binder.restoreCallingIdentity(token);
- }
- throw new IllegalStateException("Could not recompute value!");
- }
-}
diff --git a/core/java/android/app/compat/ChangeIdStateQuery.java b/core/java/android/app/compat/ChangeIdStateQuery.java
deleted file mode 100644
index 2c4c120672ab..000000000000
--- a/core/java/android/app/compat/ChangeIdStateQuery.java
+++ /dev/null
@@ -1,87 +0,0 @@
-/*
- * Copyright (C) 2020 The Android Open Source Project
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package android.app.compat;
-
-import android.annotation.IntDef;
-import android.annotation.NonNull;
-
-import java.lang.annotation.Retention;
-import java.lang.annotation.RetentionPolicy;
-import java.util.Objects;
-
-
-/**
- * A key type for caching calls to {@link com.android.internal.compat.IPlatformCompat}
- *
- * <p>For {@link com.android.internal.compat.IPlatformCompat#isChangeEnabledByPackageName}
- * and {@link com.android.internal.compat.IPlatformCompat#isChangeEnabledByUid}
- *
- * @hide
- */
-final class ChangeIdStateQuery {
-
- static final int QUERY_BY_PACKAGE_NAME = 0;
- static final int QUERY_BY_UID = 1;
- @IntDef({QUERY_BY_PACKAGE_NAME, QUERY_BY_UID})
- @Retention(RetentionPolicy.SOURCE)
- @interface QueryType {}
-
- public @QueryType int type;
- public long changeId;
- public String packageName;
- public int uid;
- public int userId;
-
- private ChangeIdStateQuery(@QueryType int type, long changeId, String packageName,
- int uid, int userId) {
- this.type = type;
- this.changeId = changeId;
- this.packageName = packageName;
- this.uid = uid;
- this.userId = userId;
- }
-
- static ChangeIdStateQuery byPackageName(long changeId, @NonNull String packageName,
- int userId) {
- return new ChangeIdStateQuery(QUERY_BY_PACKAGE_NAME, changeId, packageName, 0, userId);
- }
-
- static ChangeIdStateQuery byUid(long changeId, int uid) {
- return new ChangeIdStateQuery(QUERY_BY_UID, changeId, null, uid, 0);
- }
-
- @Override
- public boolean equals(Object other) {
- if (this == other) {
- return true;
- }
- if ((other == null) || !(other instanceof ChangeIdStateQuery)) {
- return false;
- }
- final ChangeIdStateQuery that = (ChangeIdStateQuery) other;
- return this.type == that.type
- && this.changeId == that.changeId
- && Objects.equals(this.packageName, that.packageName)
- && this.uid == that.uid
- && this.userId == that.userId;
- }
-
- @Override
- public int hashCode() {
- return Objects.hash(type, changeId, packageName, uid, userId);
- }
-}
diff --git a/core/java/android/app/compat/CompatChanges.java b/core/java/android/app/compat/CompatChanges.java
index 0d5e45f9e5d4..e289a2775b79 100644
--- a/core/java/android/app/compat/CompatChanges.java
+++ b/core/java/android/app/compat/CompatChanges.java
@@ -19,8 +19,14 @@ package android.app.compat;
import android.annotation.NonNull;
import android.annotation.SystemApi;
import android.compat.Compatibility;
+import android.content.Context;
+import android.os.Binder;
+import android.os.RemoteException;
+import android.os.ServiceManager;
import android.os.UserHandle;
+import com.android.internal.compat.IPlatformCompat;
+
/**
* CompatChanges APIs - to be used by platform code only (including mainline
* modules).
@@ -29,7 +35,6 @@ import android.os.UserHandle;
*/
@SystemApi
public final class CompatChanges {
- private static final ChangeIdStateCache QUERY_CACHE = new ChangeIdStateCache();
private CompatChanges() {}
/**
@@ -64,8 +69,17 @@ public final class CompatChanges {
*/
public static boolean isChangeEnabled(long changeId, @NonNull String packageName,
@NonNull UserHandle user) {
- return QUERY_CACHE.query(ChangeIdStateQuery.byPackageName(changeId, packageName,
- user.getIdentifier()));
+ IPlatformCompat platformCompat = IPlatformCompat.Stub.asInterface(
+ ServiceManager.getService(Context.PLATFORM_COMPAT_SERVICE));
+ final long token = Binder.clearCallingIdentity();
+ try {
+ return platformCompat.isChangeEnabledByPackageName(changeId, packageName,
+ user.getIdentifier());
+ } catch (RemoteException e) {
+ throw e.rethrowFromSystemServer();
+ } finally {
+ Binder.restoreCallingIdentity(token);
+ }
}
/**
@@ -87,7 +101,15 @@ public final class CompatChanges {
* @return {@code true} if the change is enabled for the current app.
*/
public static boolean isChangeEnabled(long changeId, int uid) {
- return QUERY_CACHE.query(ChangeIdStateQuery.byUid(changeId, uid));
+ IPlatformCompat platformCompat = IPlatformCompat.Stub.asInterface(
+ ServiceManager.getService(Context.PLATFORM_COMPAT_SERVICE));
+ final long token = Binder.clearCallingIdentity();
+ try {
+ return platformCompat.isChangeEnabledByUid(changeId, uid);
+ } catch (RemoteException e) {
+ throw e.rethrowFromSystemServer();
+ } finally {
+ Binder.restoreCallingIdentity(token);
+ }
}
-
}