From ca9e12eec9bb45f8c7f37c679fc21fd8df1b5a0b Mon Sep 17 00:00:00 2001
From: Nicholas Ambur <nambur@google.com>
Date: Wed, 22 Mar 2023 14:59:21 +0000
Subject: [PATCH] fix enable checks to check use action in LatencyTracker

Without checking the per-action enable, logging is only allowed
based on the global enable flag instead of the enable flag per action.
Checking the per-action enable flag allows for action latencies to be
rolled out independent of each other.

Test: manual verify action is logged with only action specific enable
set
Bug: 265850090

Change-Id: I4f8d21bca4a9e52fb3875e88387b8c8641f64c94
Merged-In: I4f8d21bca4a9e52fb3875e88387b8c8641f64c94
---
 core/java/com/android/internal/util/LatencyTracker.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/core/java/com/android/internal/util/LatencyTracker.java b/core/java/com/android/internal/util/LatencyTracker.java
index c933bfd5b3724..c8a8b586590b2 100644
--- a/core/java/com/android/internal/util/LatencyTracker.java
+++ b/core/java/com/android/internal/util/LatencyTracker.java
@@ -471,7 +471,7 @@ public class LatencyTracker {
      */
     public void onActionStart(@Action int action, String tag) {
         synchronized (mLock) {
-            if (!isEnabled()) {
+            if (!isEnabled(action)) {
                 return;
             }
             // skip if the action is already instrumenting.
@@ -495,7 +495,7 @@ public class LatencyTracker {
      */
     public void onActionEnd(@Action int action) {
         synchronized (mLock) {
-            if (!isEnabled()) {
+            if (!isEnabled(action)) {
                 return;
             }
             Session session = mSessions.get(action);