From 8b9c3802f60eb394dd0726e3b138dac9324517a4 Mon Sep 17 00:00:00 2001 From: Jay Sullivan Date: Fri, 3 Mar 2023 23:03:04 +0000 Subject: 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 Merged-In: Ia9ade03369de94ece045ec1433c9797e8eb8ce1e Test: CrystalBall CUJ: android.platform.test.scenario.permissions.PermissionsTest (cherry picked from commit 9bfeb112079812ef2cfbedc27bd37dd96d99d1da) --- .../com/android/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 dd06464c4699..a028ae16da2f 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( -- cgit v1.2.3-59-g8ed1b