commit c57defc3d0c032246a259081bf99cec92fa6e268
author Eugene Susla <eugenesusla@google.com> Thu Apr 09 10:41:01 2020 -0700
committer android-build-team Robot <android-build-team-robot@google.com> Thu Apr 09 18:53:37 2020 +0000
tree 05487898e243fdd0eca8889cba2e27ad3e4c9474
parent c41f4f52995e149bdead6c9954d1b851eb650f3b

Temporarily disable auto-revoke

Feature gets exposed to droidfood despite the lack of R+ apps

Bug: 153647700
Test: presubmit
Change-Id: I5481d2f0d59a5074b5f5633f451b870a9ae0707b
(cherry picked from commit 3cf8b471735f8d470268abd9f60941b10c2a793a)

src/com/android/permissioncontroller/permission/service/AutoRevokePermissions.kt

diff --git a/src/com/android/permissioncontroller/permission/service/AutoRevokePermissions.kt b/src/com/android/permissioncontroller/permission/service/AutoRevokePermissions.kt
index e3353fd..3740419 100644
--- a/src/com/android/permissioncontroller/permission/service/AutoRevokePermissions.kt
+++ b/src/com/android/permissioncontroller/permission/service/AutoRevokePermissions.kt
@@ -95,6 +95,9 @@
 private const val LOG_TAG = "AutoRevokePermissions"
 private const val DEBUG = true
 
+// TODO eugenesusla: temporarily disabled due to issues in droidfood
+private const val AUTO_REVOKE_ENABLED = false
+
 private val DEFAULT_UNUSED_THRESHOLD_MS = DAYS.toMillis(90)
 private fun getUnusedThresholdMs(context: Context) = when {
     DEBUG -> SECONDS.toMillis(1)
@@ -117,7 +120,9 @@
     PERMISSION_GRANT_REQUEST_RESULT_REPORTED__RESULT__AUTO_UNUSED_APP_PERMISSION_REVOKED
 
 private fun isAutoRevokeEnabled(context: Context): Boolean {
-    return getCheckFrequencyMs(context) > 0 && getUnusedThresholdMs(context) > 0
+    return AUTO_REVOKE_ENABLED &&
+            getCheckFrequencyMs(context) > 0 &&
+            getUnusedThresholdMs(context) > 0
 }
 
 /**