Move SideFpsController requests add call

Only add a show request to the request set if the request goes through
(overlayView == null). Adding the show request even for ignored requests
may be the source of the indicator persisting on screen when it should
be hidden, because the hide() method checks if the set is empty before
setting the overlayView to null.

Flag: NONE
Fixes: 338778802
Test: None (cannot reproduce reported issue)
Change-Id: Ic50c0709f4c62385a2aa941f7b3e1221e663a1d2
Merged-In: Ied1e743db40ecffc0ed07aa506eaf9be82c57a0a

1 files changed, 1 insertions, 1 deletions

diff --git a/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt b/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt
index 458cdeaf0208..19e6a86c6e2f 100644
--- a/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt
+++ b/packages/SystemUI/src/com/android/systemui/biometrics/SideFpsController.kt
@@ -218,12 +218,12 @@ constructor(
     ) {
         SideFpsControllerRefactor.assertInLegacyMode()
         if (!displayStateInteractor.isInRearDisplayMode.value) {
-            requests.add(request)
             mainExecutor.execute {
                 if (overlayView == null) {
                     traceSection(
                         "SideFpsController#show(request=${request.name}, reason=$reason)"
                     ) {
+                        requests.add(request)
                         createOverlayForDisplay(reason)
                     }
                 } else {