From 59d34b59482ce51a29de44bc5c90bd1d91f26e23 Mon Sep 17 00:00:00 2001 From: Nicolo' Mazzucato Date: Mon, 20 Feb 2023 17:12:34 +0000 Subject: [PATCH] Show traces when side fps is shown or hidden (with reasons) Bug: 262322084 Test: perfetto trace Change-Id: I1fc36d9af72d529219d6d7742a7efd0bae4f36bc --- .../com/android/systemui/biometrics/SideFpsController.kt | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt b/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt index 6c490780b79a9..13ecc8f559dcb 100644 --- a/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt +++ b/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt @@ -64,6 +64,7 @@ import com.android.systemui.flags.Flags import com.android.systemui.keyguard.domain.interactor.AlternateBouncerInteractor import com.android.systemui.recents.OverviewProxyService import com.android.systemui.util.concurrency.DelayableExecutor +import com.android.systemui.util.traceSection import java.io.PrintWriter import javax.inject.Inject import kotlinx.coroutines.CoroutineScope @@ -211,7 +212,9 @@ constructor( requests.add(request) mainExecutor.execute { if (overlayView == null) { - createOverlayForDisplay(reason) + traceSection("SideFpsController#show(request=${request.name}, reason=$reason") { + createOverlayForDisplay(reason) + } } else { Log.v(TAG, "overlay already shown") } @@ -223,7 +226,7 @@ constructor( requests.remove(request) mainExecutor.execute { if (requests.isEmpty()) { - overlayView = null + traceSection("SideFpsController#hide(${request.name}") { overlayView = null } } } }