From 9bfeb112079812ef2cfbedc27bd37dd96d99d1da Mon Sep 17 00:00:00 2001
From: Jay Sullivan <jaysullivan@google.com>
Date: Fri, 3 Mar 2023 23:03:04 +0000
Subject: [PATCH] Change trace tag for AppOpsCheckingServiceTracingDecorator

"Trace.TRACE_TAG_SYSTEM_SERVER" is used here as a trace category. This is probably fine, but, CrystalBall itself isn't currently configured to log this category.

As a workaround, use the trace tag "Trace.TRACE_TAG_ACTIVITY_MANAGER" instead.

Later, we might want to change this back to "Trace.TRACE_TAG_SYSTEM_SERVER" (or even create a new tag for permissions), and configure CrystalBall to support this category.

Bug: 252887420
Change-Id: Ia9ade03369de94ece045ec1433c9797e8eb8ce1e
Test: CrystalBall CUJ: android.platform.test.scenario.permissions.PermissionsTest
---
 .../server/appop/AppOpsCheckingServiceTracingDecorator.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/services/core/java/com/android/server/appop/AppOpsCheckingServiceTracingDecorator.java b/services/core/java/com/android/server/appop/AppOpsCheckingServiceTracingDecorator.java
index dd06464c46996..a028ae16da2f3 100644
--- a/services/core/java/com/android/server/appop/AppOpsCheckingServiceTracingDecorator.java
+++ b/services/core/java/com/android/server/appop/AppOpsCheckingServiceTracingDecorator.java
@@ -32,7 +32,7 @@ import java.io.PrintWriter;
  * Trace.traceEnd. These traces are used for performance testing.
  */
 public class AppOpsCheckingServiceTracingDecorator implements AppOpsCheckingServiceInterface {
-    private static final long TRACE_TAG = Trace.TRACE_TAG_SYSTEM_SERVER;
+    private static final long TRACE_TAG = Trace.TRACE_TAG_ACTIVITY_MANAGER;
     private final AppOpsCheckingServiceInterface mService;
 
     AppOpsCheckingServiceTracingDecorator(