Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Change trace tag for AppOpsCheckingServiceTracingDecorator

Browse Source 
"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 9bfeb11207)
This commit is contained in:
Jay Sullivan
2023-03-03 23:03:04 +00:00
parent 6789d00256
commit 8b9c3802f6
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/core/java/com/android/server/appop/AppOpsCheckingServiceTracingDecorator.java
View File

@@ -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(