From b01d42ea8ccaa06208fc842ee2353313f6bba426 Mon Sep 17 00:00:00 2001 From: Makoto Onuki Date: Mon, 8 May 2023 11:14:55 -0700 Subject: [PATCH] Extract WTF to a separate method for easier search Bug: 276963716 Test: build Change-Id: Ifc6b2509c0f1e6c5f115f4bf050c8626763519a9 --- .../core/java/com/android/server/am/ActiveServices.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/am/ActiveServices.java b/services/core/java/com/android/server/am/ActiveServices.java index ca50af8075c66..96766a20c8030 100644 --- a/services/core/java/com/android/server/am/ActiveServices.java +++ b/services/core/java/com/android/server/am/ActiveServices.java @@ -2326,7 +2326,7 @@ public final class ActiveServices { && (r.getConnections().size() > 0) && (r.mDebugWhileInUseReasonInBindService != r.mDebugWhileInUseReasonInStartForeground)) { - Slog.wtf(TAG, "FGS while-in-use changed (b/276963716): old=" + logWhileInUseChangeWtf("FGS while-in-use changed (b/276963716): old=" + reasonCodeToString(r.mDebugWhileInUseReasonInBindService) + " new=" + reasonCodeToString(r.mDebugWhileInUseReasonInStartForeground) @@ -2589,6 +2589,13 @@ public final class ActiveServices { } } + /** + * It just does a wtf, but extracted to a method, so we can do a signature search on pitot. + */ + private void logWhileInUseChangeWtf(String message) { + Slog.wtf(TAG, message); + } + private boolean withinFgsDeferRateLimit(ServiceRecord sr, final long now) { // If we're still within the service's deferral period, then by definition // deferral is not rate limited.