Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 850b07d4 authored by Caitlin Shkuratov's avatar Caitlin Shkuratov
Browse files

[SB] Add table logs in HomeStatusBarViewModel for keyguard state. 

It will make bugs like b/430055269 easier to debug, since we only need
to look at 1 bugreport view not 2.

Bug: 430055269
Flag: EXEMPT logging change
Test: lock & unlock a few times then dump HomeStatusBarViewModel[0] ->
verify keyguard state is logged

Change-Id: I5841d3efc8fda6258f18e15b7e6875517b8558ce
parent a59b570c

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/shared/ui/viewmodel/HomeStatusBarViewModel.kt

+13 −1
Original line number Diff line number Diff line
@@ -31,6 +31,7 @@ import com.android.systemui.display.dagger.SystemUIDisplaySubcomponent.DisplayId 
import com.android.systemui.keyguard.domain.interactor.KeyguardInteractor

import com.android.systemui.keyguard.domain.interactor.KeyguardTransitionInteractor

import com.android.systemui.keyguard.shared.model.Edge

import com.android.systemui.keyguard.shared.model.KeyguardState

import com.android.systemui.keyguard.shared.model.KeyguardState.DREAMING

import com.android.systemui.keyguard.shared.model.KeyguardState.GONE

import com.android.systemui.keyguard.shared.model.KeyguardState.LOCKSCREEN
@@ -98,6 +99,7 @@ import kotlinx.coroutines.flow.filter 
import kotlinx.coroutines.flow.flowOf

import kotlinx.coroutines.flow.flowOn

import kotlinx.coroutines.flow.map

import kotlinx.coroutines.flow.onEach

import kotlinx.coroutines.flow.stateIn



/**
@@ -407,12 +409,21 @@ constructor( 
            source = darkIconInteractor.isAreaDark(thisDisplayId),

        )



    private val currentKeyguardState: Flow<KeyguardState> =

        keyguardTransitionInteractor.currentKeyguardState.onEach {

            tableLogger.logChange(

                columnName = COL_KEYGUARD_STATE,

                value = it.name,

                isInitial = false,

            )

        }



    /**

     * True if the current SysUI state can show the home status bar (aka this status bar), and false

     * if we shouldn't be showing any part of the home status bar.

     */

    private val isHomeScreenStatusBarAllowedLegacy: Flow<Boolean> =

        combine(keyguardTransitionInteractor.currentKeyguardState, isShadeVisibleOnThisDisplay) {

        combine(currentKeyguardState, isShadeVisibleOnThisDisplay) {

                currentKeyguardState,

                isShadeVisibleOnThisDisplay ->

                (currentKeyguardState == GONE || currentKeyguardState == OCCLUDED) &&
@@ -681,6 +692,7 @@ constructor( 
        private const val COL_ALLOWED_LEGACY = "allowedLegacy"

        private const val COL_ALLOWED_BY_SCENE = "allowedByScene"

        private const val COL_SHADE_EXPANDED_ENOUGH = "shadeExpandedEnough"

        private const val COL_KEYGUARD_STATE = "keyguardState"

        private const val COL_NOTIF_LIGHTS_OUT = "notifLightsOut"

        private const val COL_SHOW_OPERATOR_NAME = "showOperatorName"

        private const val COL_VISIBLE = "visible"