summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nicholas Ambur <nambur@google.com> 2023-03-22 14:59:21 +0000
committer Nicholas Ambur <nambur@google.com> 2023-03-22 20:56:07 +0000
commitca9e12eec9bb45f8c7f37c679fc21fd8df1b5a0b (patch)
treeabf1da17533f1162149a28d535ee5acdac6980a4
parentc37e3adeffd4e79193c9fc23f2795a043d673ab9 (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.java4
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);