From 543360eadbc97f6af9d1496130d1d6ef2f76ef1d Mon Sep 17 00:00:00 2001
From: rushiiMachine <33725716+rushiiMachine@users.noreply.github.com>
Date: Wed, 22 Jan 2025 18:53:36 -0800
Subject: [PATCH] feat(ManifestPatcher): remove `vmSafeMode` attribute

Unnecessary since GrapheneOS didn't enable aggressive AOT until recently (which was already API >= 29) and is that branch covered by another workaround.
---
 .../manager/patcher/util/ManifestPatcher.kt   | 25 -------------------
 .../kotlin/com/aliucord/manager/util/Utils.kt |  8 ------
 2 files changed, 33 deletions(-)

diff --git a/app/src/main/kotlin/com/aliucord/manager/patcher/util/ManifestPatcher.kt b/app/src/main/kotlin/com/aliucord/manager/patcher/util/ManifestPatcher.kt
index 45bbfd2..f48e971 100644
--- a/app/src/main/kotlin/com/aliucord/manager/patcher/util/ManifestPatcher.kt
+++ b/app/src/main/kotlin/com/aliucord/manager/patcher/util/ManifestPatcher.kt
@@ -7,7 +7,6 @@ package com.aliucord.manager.patcher.util
 
 import android.Manifest
 import android.os.Build
-import com.aliucord.manager.util.isGrapheneOS
 import pxb.android.axml.*
 
 object ManifestPatcher {
@@ -100,14 +99,12 @@ object ManifestPatcher {
                             ) {
                                 private var addDebuggable = debuggable
                                 private var addLegacyStorage = true
-                                private var addVmSafeMode = true
                                 private var addUseEmbeddedDex = true
                                 private var addExtractNativeLibs = true
 
                                 override fun attr(ns: String?, name: String, resourceId: Int, type: Int, value: Any?) {
                                     if (name == NETWORK_SECURITY_CONFIG) return
                                     if (name == REQUEST_LEGACY_EXTERNAL_STORAGE) addLegacyStorage = false
-                                    if (name == VM_SAFE_MODE) addVmSafeMode = false
                                     if (name == USE_EMBEDDED_DEX) addUseEmbeddedDex = false
                                     if (name == EXTRACT_NATIVE_LIBS) addExtractNativeLibs = false
                                     if (name == DEBUGGABLE) addDebuggable = false
@@ -153,10 +150,6 @@ object ManifestPatcher {
                                     if (Build.VERSION.SDK_INT >= 29 && addUseEmbeddedDex) {
                                         super.attr(ANDROID_NAMESPACE, USE_EMBEDDED_DEX, android.R.attr.useEmbeddedDex, TYPE_INT_BOOLEAN, 1)
                                     }
-                                    // GrapheneOS also forces AOT, disable it in a worse way if below API 29
-                                    else if (isGrapheneOS() && addVmSafeMode) {
-                                        super.attr(ANDROID_NAMESPACE, VM_SAFE_MODE, android.R.attr.vmSafeMode, TYPE_INT_BOOLEAN, 1)
-                                    }
 
                                     if (addExtractNativeLibs) super.attr(
                                         ANDROID_NAMESPACE,
@@ -179,24 +172,6 @@ object ManifestPatcher {
         return writer.toByteArray()
     }
 
-    fun renamePackage(
-        manifestBytes: ByteArray,
-        packageName: String,
-    ): ByteArray {
-        val reader = AxmlReader(manifestBytes)
-        val writer = AxmlWriter()
-
-        reader.accept(
-            object : AxmlVisitor(writer) {
-                override fun child(ns: String?, name: String?): ReplaceAttrsVisitor {
-                    return ReplaceAttrsVisitor(super.child(ns, name), mapOf("package" to packageName))
-                }
-            }
-        )
-
-        return writer.toByteArray()
-    }
-
     private open class ReplaceAttrsVisitor(
         nv: NodeVisitor,
         private val attrs: Map<String, Any>,
diff --git a/app/src/main/kotlin/com/aliucord/manager/util/Utils.kt b/app/src/main/kotlin/com/aliucord/manager/util/Utils.kt
index d1f814b..70c98f0 100644
--- a/app/src/main/kotlin/com/aliucord/manager/util/Utils.kt
+++ b/app/src/main/kotlin/com/aliucord/manager/util/Utils.kt
@@ -76,14 +76,6 @@ fun isMiui(): Boolean {
         ?.isNotEmpty() ?: false
 }
 
-
-/**
- * Checks whether this device is possibly running GrapheneOS.
- */
-fun isGrapheneOS(): Boolean {
-    return Build.USER == "grapheneos" || Build.HOST == "grapheneos"
-}
-
 /**
  * Gets a system property from build.prop
  */