From 795022cc68becd94bdc67b23ff8855cec114c619 Mon Sep 17 00:00:00 2001 From: Yi-an Chen Date: Tue, 29 Oct 2024 20:45:54 +0000 Subject: Revert "Clean up unused lib/rules in build file" This reverts commit 4e5889e2d0d1aab829ce72c0695a481fcb34657e. Reason for revert: Causing b/375725141 and resulting in method name conflicts for MTS on lower platforms that don't have jarjar_prefix Change-Id: I6987931531cd15acbede0ed2b741ad7f12cbd8cf --- .../java/com/android/role/controller/model/RoleParser.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'PermissionController/role-controller/java') diff --git a/PermissionController/role-controller/java/com/android/role/controller/model/RoleParser.java b/PermissionController/role-controller/java/com/android/role/controller/model/RoleParser.java index a0705cd5e..6d5cf69bd 100644 --- a/PermissionController/role-controller/java/com/android/role/controller/model/RoleParser.java +++ b/PermissionController/role-controller/java/com/android/role/controller/model/RoleParser.java @@ -1106,7 +1106,7 @@ public class RoleParser { + ">"); return fallbackValue; } - String className = applyJarjarTransformIfNeeded(value.substring(0, lastDotIndex)); + String className = applyJarjarTransform(value.substring(0, lastDotIndex)); String methodName = value.substring(lastDotIndex + 1); Method method; try { @@ -1155,16 +1155,18 @@ public class RoleParser { }; } - // LINT.IfChange(applyJarjarTransformIfNeeded) + // LINT.IfChange(applyJarjarTransform) /** * Simulate the jarjar transform that should happen on the class name. *

* Currently this only handles the {@code Flags} classes for feature flagging. */ @NonNull - private String applyJarjarTransformIfNeeded(@NonNull String className) { - if (className.endsWith(".Flags") && Objects.equals(mContext.getPackageName(), "android")) { - return "com.android.permission.jarjar." + className; + private String applyJarjarTransform(@NonNull String className) { + if (className.endsWith(".Flags")) { + String jarjarPrefix = Objects.equals(mContext.getPackageName(), "android") + ? "com.android.permission.jarjar." : "com.android.permissioncontroller.jarjar."; + return jarjarPrefix + className; } return className; } -- cgit v1.2.3-59-g8ed1b