diff options
| author | 2023-03-22 14:59:21 +0000 | |
|---|---|---|
| committer | 2023-03-22 20:56:07 +0000 | |
| commit | ca9e12eec9bb45f8c7f37c679fc21fd8df1b5a0b (patch) | |
| tree | abf1da17533f1162149a28d535ee5acdac6980a4 | |
| parent | c37e3adeffd4e79193c9fc23f2795a043d673ab9 (diff) | |
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
| -rw-r--r-- | core/java/com/android/internal/util/LatencyTracker.java | 4 |
1 files 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 c933bfd5b372..c8a8b586590b 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); |