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

Commit da2ad087 authored by Chris Göllner's avatar Chris Göllner Committed by Android (Google) Code Review
Browse files

Revert "Use display aware SystemBarUtilsState and ConfigurationState" 

This reverts commit c5a7d7e9.

Reason for revert: b/426094163 and b/426452135
Bug: 419552860
Fixes: 426452135
Fixes: 426094163

Change-Id: I9d6614ad3d7810440cfb841e01100f1bd81fbcb5
parent c5a7d7e9

packages/SystemUI/compose/features/src/com/android/systemui/keyguard/ui/composable/element/AodNotificationIconsElement.kt

+1 −1
Original line number Diff line number Diff line
@@ -61,7 +61,7 @@ constructor( 
    private val iconBindingFailureTracker: StatusBarIconViewBindingFailureTracker,

    private val nicAodViewModel: NotificationIconContainerAlwaysOnDisplayViewModel,

    private val nicAodIconViewStore: AlwaysOnDisplayNotificationIconViewStore,

    @ShadeDisplayAware private val systemBarUtilsState: SystemBarUtilsState,

    private val systemBarUtilsState: SystemBarUtilsState,

    private val keyguardClockViewModel: KeyguardClockViewModel,

) {

packages/SystemUI/src/com/android/systemui/display/dagger/SystemUIDisplaySubcomponent.kt

+0 −6
Original line number Diff line number Diff line
@@ -16,12 +16,10 @@ 


package com.android.systemui.display.dagger



import com.android.systemui.common.ui.ConfigurationState

import com.android.systemui.display.dagger.SystemUIDisplaySubcomponent.PerDisplaySingleton

import com.android.systemui.display.data.repository.DisplayStateRepository

import com.android.systemui.display.domain.interactor.DisplayStateInteractor

import com.android.systemui.statusbar.domain.interactor.StatusBarIconRefreshInteractor

import com.android.systemui.statusbar.ui.SystemBarUtilsState

import dagger.BindsInstance

import dagger.Subcomponent

import javax.inject.Qualifier
@@ -51,10 +49,6 @@ interface SystemUIDisplaySubcomponent { 


    @get:DisplayAware val lifecycleListeners: Set<LifecycleListener>



    @get:DisplayAware val systemBarUtilsState: SystemBarUtilsState



    @get:DisplayAware val configurationState: ConfigurationState



    @Subcomponent.Factory

    interface Factory {

        fun create(@BindsInstance @DisplayId displayId: Int): SystemUIDisplaySubcomponent

packages/SystemUI/src/com/android/systemui/keyguard/ui/view/layout/sections/AodNotificationIconsSection.kt

+1 −1
Original line number Diff line number Diff line
@@ -54,7 +54,7 @@ constructor( 
    private val iconBindingFailureTracker: StatusBarIconViewBindingFailureTracker,

    private val nicAodViewModel: NotificationIconContainerAlwaysOnDisplayViewModel,

    private val nicAodIconViewStore: AlwaysOnDisplayNotificationIconViewStore,

    @ShadeDisplayAware private val systemBarUtilsState: SystemBarUtilsState,

    private val systemBarUtilsState: SystemBarUtilsState,

    private val rootViewModel: KeyguardRootViewModel,

    private val shadeModeInteractor: ShadeModeInteractor,

) : KeyguardSection() {

packages/SystemUI/src/com/android/systemui/shade/ShadeDisplayAwareModule.kt

+0 −12
Original line number Diff line number Diff line
@@ -59,7 +59,6 @@ import com.android.systemui.statusbar.phone.ConfigurationForwarder 
import com.android.systemui.statusbar.phone.domain.interactor.ShadeDarkIconInteractor

import com.android.systemui.statusbar.phone.domain.interactor.ShadeDarkIconInteractorImpl

import com.android.systemui.statusbar.policy.ConfigurationController

import com.android.systemui.statusbar.ui.SystemBarUtilsState

import com.android.systemui.utils.windowmanager.WindowManagerProvider

import com.android.window.flags.Flags

import dagger.Module
@@ -211,17 +210,6 @@ object ShadeDisplayAwareModule { 
        }

    }



    @SysUISingleton

    @Provides

    @ShadeDisplayAware

    fun shadeDisplayAwareSystemBarUtilsState(

        @ShadeDisplayAware context: Context,

        @ShadeDisplayAware configurationController: ConfigurationController,

        factory: SystemBarUtilsState.Factory,

    ): SystemBarUtilsState {

        return factory.create(context, configurationController)

    }



    @SysUISingleton

    @Provides

    @ShadeDisplayAware

packages/SystemUI/src/com/android/systemui/statusbar/dagger/StatusBarPerDisplayModule.kt

+1 −27
Original line number Diff line number Diff line
@@ -16,9 +16,6 @@ 


package com.android.systemui.statusbar.dagger



import android.content.Context

import com.android.systemui.common.ui.ConfigurationState

import com.android.systemui.common.ui.ConfigurationStateImpl

import com.android.systemui.display.dagger.SystemUIDisplaySubcomponent

import com.android.systemui.display.dagger.SystemUIDisplaySubcomponent.DisplayAware

import com.android.systemui.display.dagger.SystemUIDisplaySubcomponent.PerDisplaySingleton
@@ -26,7 +23,6 @@ import com.android.systemui.statusbar.data.repository.StatusBarConfigurationCont 
import com.android.systemui.statusbar.data.repository.StatusBarConfigurationControllerStore

import com.android.systemui.statusbar.domain.interactor.StatusBarIconRefreshInteractor

import com.android.systemui.statusbar.domain.interactor.StatusBarIconRefreshInteractorImpl

import com.android.systemui.statusbar.ui.SystemBarUtilsState

import dagger.Binds

import dagger.Module

import dagger.Provides
@@ -57,33 +53,11 @@ interface StatusBarPerDisplayModule { 
        @PerDisplaySingleton

        @DisplayAware

        fun provideStatusBarConfigurationController(

            @DisplayAware displayId: Int,

            @SystemUIDisplaySubcomponent.DisplayAware displayId: Int,

            configurationControllerStore: StatusBarConfigurationControllerStore,

        ): StatusBarConfigurationController {

            return configurationControllerStore.forDisplay(displayId)

                ?: error("No configuration controller for display $displayId")

        }



        @Provides

        @PerDisplaySingleton

        @DisplayAware

        fun systemBarUtilsState(

            @DisplayAware context: Context,

            @DisplayAware configurationController: StatusBarConfigurationController,

            factory: SystemBarUtilsState.Factory,

        ): SystemBarUtilsState {

            return factory.create(context, configurationController)

        }



        @Provides

        @PerDisplaySingleton

        @DisplayAware

        fun configurationState(

            configStateFactory: ConfigurationStateImpl.Factory,

            @DisplayAware configurationController: StatusBarConfigurationController,

            @DisplayAware context: Context,

        ): ConfigurationState {

            return configStateFactory.create(context, configurationController)

        }

    }

}