Revert "Fix work profile reinflations when shade window moves between displays" (848759d0) · Commits · e / os / android_frameworks_base

packages/SystemUI/multivalentTests/src/com/android/systemui/display/data/repository/DisplayWindowPropertiesRepositoryImplTest.kt

+4 −2

Original line number	Diff line number	Diff line
		@@ -63,6 +63,8 @@ class DisplayWindowPropertiesRepositoryImplTest : SysuiTestCase() {
		DisplayWindowPropertiesRepositoryImpl(
		kosmos.applicationCoroutineScope,
		applicationContext,
		applicationWindowManager,
		kosmos.layoutInflater,
		fakeDisplayRepository,
		)
		}
		@@ -96,12 +98,12 @@ class DisplayWindowPropertiesRepositoryImplTest : SysuiTestCase() {
		}

		@Test
		fun get_defaultDisplayId_doesNotReturnDefaultProperties() =
		fun get_defaultDisplayId_returnsDefaultProperties() =
		testScope.runTest {
		val displayContext = repo.get(DEFAULT_DISPLAY_ID, WINDOW_TYPE_FOO)

		assertThat(displayContext)
		.isNotEqualTo(
		.isEqualTo(
		DisplayWindowProperties(
		displayId = DEFAULT_DISPLAY_ID,
		windowType = WINDOW_TYPE_FOO,

packages/SystemUI/src/com/android/systemui/display/data/repository/DisplayWindowPropertiesRepository.kt

+25 −17

Original line number	Diff line number	Diff line
		@@ -58,6 +58,8 @@ class DisplayWindowPropertiesRepositoryImpl
		constructor(
		@Background private val backgroundApplicationScope: CoroutineScope,
		private val globalContext: Context,
		private val globalWindowManager: WindowManager,
		private val globalLayoutInflater: LayoutInflater,
		private val displayRepository: DisplayRepository,
		) : DisplayWindowPropertiesRepository, CoreStartable {

		@@ -93,6 +95,17 @@ constructor(

		private fun create(display: Display, windowType: Int): DisplayWindowProperties? {
		val displayId = display.displayId
		return if (displayId == Display.DEFAULT_DISPLAY) {
		// For the default display, we can just reuse the global/application properties.
		// Creating a window context is expensive, therefore we avoid it.
		DisplayWindowProperties(
		displayId = displayId,
		windowType = windowType,
		context = globalContext,
		windowManager = globalWindowManager,
		layoutInflater = globalLayoutInflater,
		)
		} else {
		val context = createWindowContext(display, windowType)
		if (context.displayId != display.displayId) {
		Log.e(
		@@ -105,13 +118,8 @@ constructor(
		@SuppressLint("NonInjectedService") // Need to manually get the service
		val windowManager = WindowManagerUtils.getWindowManager(context)
		val layoutInflater = LayoutInflater.from(context)
		return DisplayWindowProperties(
		displayId,
		windowType,
		context,
		windowManager,
		layoutInflater,
		)
		DisplayWindowProperties(displayId, windowType, context, windowManager, layoutInflater)
		}
		}

		private fun createWindowContext(display: Display, windowType: Int): Context =

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

deleted100644 → 0

+0 −22

Original line number	Diff line number	Diff line
		/*
		* Copyright (C) 2025 The Android Open Source Project
		*
		* Licensed under the Apache License, Version 2.0 (the "License");
		* you may not use this file except in compliance with the License.
		* You may obtain a copy of the License at
		*
		* http://www.apache.org/licenses/LICENSE-2.0
		*
		* Unless required by applicable law or agreed to in writing, software
		* distributed under the License is distributed on an "AS IS" BASIS,
		* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
		* See the License for the specific language governing permissions and
		* limitations under the License.
		*/

		package com.android.systemui.statusbar.dagger

		import javax.inject.Qualifier

		/** Qualifier that annotates instances related to the default display statusbar. */
		@Qualifier @Retention(AnnotationRetention.RUNTIME) annotation class StatusBarMain

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

+0 −18

Original line number	Diff line number	Diff line
		@@ -17,15 +17,11 @@
		package com.android.systemui.statusbar.dagger

		import android.content.Context
		import android.view.Display
		import android.view.WindowManager.LayoutParams.TYPE_STATUS_BAR
		import com.android.systemui.CameraProtectionLoader
		import com.android.systemui.CoreStartable
		import com.android.systemui.SysUICutoutProvider
		import com.android.systemui.SysUICutoutProviderImpl
		import com.android.systemui.dagger.SysUISingleton
		import com.android.systemui.dagger.qualifiers.Main
		import com.android.systemui.display.data.repository.DisplayWindowPropertiesRepositoryImpl
		import com.android.systemui.log.LogBuffer
		import com.android.systemui.log.LogBufferFactory
		import com.android.systemui.statusbar.chips.sharetoapp.ui.viewmodel.ShareToAppChipViewModel
		@@ -207,19 +203,5 @@ interface StatusBarModule {
		): StatusBarContentInsetsViewModel {
		return StatusBarContentInsetsViewModel(insetsProvider)
		}

		@Provides
		@StatusBarMain
		fun provideDefaultStatusBarContext(
		repoLazy: Lazy<DisplayWindowPropertiesRepositoryImpl>,
		@Main appContext: Context,
		): Context {
		return if (StatusBarConnectedDisplays.isEnabled) {
		return repoLazy.get().get(Display.DEFAULT_DISPLAY, TYPE_STATUS_BAR)?.context
		?: appContext
		} else {
		appContext
		}
		}
		}
		}

packages/SystemUI/src/com/android/systemui/statusbar/notification/icon/IconBuilder.kt

+2 −3

Original line number	Diff line number	Diff line
		@@ -18,19 +18,18 @@ package com.android.systemui.statusbar.notification.icon

		import android.app.Notification
		import android.content.Context
		import com.android.systemui.dagger.qualifiers.Main
		import com.android.systemui.statusbar.StatusBarIconView
		import com.android.systemui.statusbar.dagger.StatusBarMain
		import com.android.systemui.statusbar.notification.collection.BundleEntry
		import com.android.systemui.statusbar.notification.collection.NotificationEntry
		import com.android.systemui.statusbar.notification.contentDescForNotification
		import javax.inject.Inject

		/** Testable wrapper around Context. */
		class IconBuilder @Inject constructor(@StatusBarMain private val context: Context) {
		class IconBuilder @Inject constructor(@Main private val context: Context) {
		@JvmOverloads
		fun createIconView(
		entry: NotificationEntry,
		// TODO b/362720336: remove all usages of this without a provided context.
		context: Context = this.context,
		): StatusBarIconView {
		return StatusBarIconView(