From 6d57fc316d1c1b9131099ceaf6a1e4dfb7976eee Mon Sep 17 00:00:00 2001 From: Jeff Sharkey Date: Wed, 2 Nov 2022 12:12:46 -0600 Subject: [PATCH] BroadcastQueue: omit overly noisy trace event. This method is currently called far too often, and pollutes trace output. We have some ideas on how to improve in the future, but remove the tracing for now. Bug: 253906105 Test: atest FrameworksMockingServicesTests:BroadcastRecordTest Test: atest FrameworksMockingServicesTests:BroadcastQueueTest Test: atest FrameworksMockingServicesTests:BroadcastQueueModernImplTest Change-Id: I414a55fea626d8f2706603fa6709bbf6f087c758 --- .../java/com/android/server/am/BroadcastQueueModernImpl.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/services/core/java/com/android/server/am/BroadcastQueueModernImpl.java b/services/core/java/com/android/server/am/BroadcastQueueModernImpl.java index fdad729e5005e..ec30353927f97 100644 --- a/services/core/java/com/android/server/am/BroadcastQueueModernImpl.java +++ b/services/core/java/com/android/server/am/BroadcastQueueModernImpl.java @@ -320,7 +320,6 @@ class BroadcastQueueModernImpl extends BroadcastQueue { return; } - final int cookie = traceBegin("updateRunnableList"); final boolean wantQueue = queue.isRunnable(); final boolean inQueue = (queue == mRunnableHead) || (queue.runnableAtPrev != null) || (queue.runnableAtNext != null); @@ -347,8 +346,6 @@ class BroadcastQueueModernImpl extends BroadcastQueue { if (queue.isEmpty() && !queue.isActive() && !queue.isProcessWarm()) { removeProcessQueue(queue.processName, queue.uid); } - - traceEnd(cookie); } /**