From 795022cc68becd94bdc67b23ff8855cec114c619 Mon Sep 17 00:00:00 2001
From: Yi-an Chen <theianchen@google.com>
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
---
 PermissionController/Android.bp                    |  8 +++++++
 PermissionController/jarjar-rules.txt              | 25 ++++++++++++++++++++++
 .../android/role/controller/model/RoleParser.java  | 12 ++++++-----
 PermissionController/tests/mocking/Android.bp      |  3 +++
 4 files changed, 43 insertions(+), 5 deletions(-)
 create mode 100644 PermissionController/jarjar-rules.txt

(limited to 'PermissionController')

diff --git a/PermissionController/Android.bp b/PermissionController/Android.bp
index 5c238806e..596b2dbb5 100644
--- a/PermissionController/Android.bp
+++ b/PermissionController/Android.bp
@@ -72,6 +72,11 @@ java_library {
     ],
 }
 
+filegroup {
+    name: "PermissionController-jarjar-rules",
+    srcs: ["jarjar-rules.txt"],
+}
+
 android_library {
     name: "PermissionController-lib",
     sdk_version: "system_current",
@@ -167,6 +172,9 @@ android_library {
         "//apex_available:platform",
         "com.android.permission",
     ],
+
+    // TODO(b/313706381): Remove jarjar once flagging lib is fixed
+    jarjar_rules: ":PermissionController-jarjar-rules",
 }
 
 android_app {
diff --git a/PermissionController/jarjar-rules.txt b/PermissionController/jarjar-rules.txt
new file mode 100644
index 000000000..7bfda1ab1
--- /dev/null
+++ b/PermissionController/jarjar-rules.txt
@@ -0,0 +1,25 @@
+# You may bypass this Gerrit IfThisThenThat Lint if your change doesn't affect
+# RoleParser.applyJarjarTransform(), by adding NO_IFTTT=reason to your commit
+# message.
+# LINT.IfChange
+rule android.app.appfunctions.flags.*FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.app.appfunctions.flags.FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.app.appfunctions.flags.FeatureFlags com.android.permissioncontroller.jarjar.@0
+rule android.app.appfunctions.flags.Flags com.android.permissioncontroller.jarjar.@0
+rule android.companion.virtualdevice.flags.*FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.companion.virtualdevice.flags.FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.companion.virtualdevice.flags.FeatureFlags com.android.permissioncontroller.jarjar.@0
+rule android.companion.virtualdevice.flags.Flags com.android.permissioncontroller.jarjar.@0
+rule android.content.pm.*FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.content.pm.FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.content.pm.FeatureFlags com.android.permissioncontroller.jarjar.@0
+rule android.content.pm.Flags com.android.permissioncontroller.jarjar.@0
+rule android.permission.flags.*FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.permission.flags.FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.permission.flags.FeatureFlags com.android.permissioncontroller.jarjar.@0
+rule android.permission.flags.Flags com.android.permissioncontroller.jarjar.@0
+rule android.os.*FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.os.FeatureFlags* com.android.permissioncontroller.jarjar.@0
+rule android.os.FeatureFlags com.android.permissioncontroller.jarjar.@0
+rule android.os.Flags com.android.permissioncontroller.jarjar.@0
+# LINT.ThenChange(PermissionController/role-controller/java/com/android/role/controller/model/RoleParser.java:applyJarjarTransform)
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.
      * <p>
      * 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;
     }
diff --git a/PermissionController/tests/mocking/Android.bp b/PermissionController/tests/mocking/Android.bp
index a541f4577..37851b2bb 100644
--- a/PermissionController/tests/mocking/Android.bp
+++ b/PermissionController/tests/mocking/Android.bp
@@ -86,4 +86,7 @@ android_test {
     ],
 
     kotlincflags: ["-Xjvm-default=all"],
+
+    // TODO(b/313706381): Remove jarjar once flagging lib is fixed
+    jarjar_rules: ":PermissionController-jarjar-rules",
 }
-- 
cgit v1.2.3-59-g8ed1b