From 529fba976e7a9093048240d855b8d889c8328dc3 Mon Sep 17 00:00:00 2001 From: Florence Yang Date: Wed, 16 Nov 2022 01:09:17 +0000 Subject: Enable Region Sampling Flag for Teamfood Enable teamfood for region sampling text protection feature that's hidden behind a flag. This is in hopes of having more people test the feature to catch bugs and get more feedback. Bug: 202758428 Test: manual Change-Id: If29b183114c06ab90edaf78765a4c8eb2d8c6bb6 --- .../SystemUI/src/com/android/keyguard/ClockEventController.kt | 9 +++++++-- packages/SystemUI/src/com/android/systemui/flags/Flags.kt | 2 +- .../statusbar/lockscreen/LockscreenSmartspaceController.kt | 4 +++- 3 files changed, 11 insertions(+), 4 deletions(-) diff --git a/packages/SystemUI/src/com/android/keyguard/ClockEventController.kt b/packages/SystemUI/src/com/android/keyguard/ClockEventController.kt index 87e9d5630b74..8f38e5800015 100644 --- a/packages/SystemUI/src/com/android/keyguard/ClockEventController.kt +++ b/packages/SystemUI/src/com/android/keyguard/ClockEventController.kt @@ -15,6 +15,7 @@ */ package com.android.keyguard +import android.app.WallpaperManager import android.content.BroadcastReceiver import android.content.Context import android.content.Intent @@ -100,9 +101,13 @@ open class ClockEventController @Inject constructor( private val regionSamplingEnabled = featureFlags.isEnabled(REGION_SAMPLING) private fun updateColors() { + if (regionSamplingEnabled && smallRegionSampler != null && largeRegionSampler != null) { - smallClockIsDark = smallRegionSampler!!.currentRegionDarkness().isDark - largeClockIsDark = largeRegionSampler!!.currentRegionDarkness().isDark + val wallpaperManager = WallpaperManager.getInstance(context) + if (!wallpaperManager.lockScreenWallpaperExists()) { + smallClockIsDark = smallRegionSampler!!.currentRegionDarkness().isDark + largeClockIsDark = largeRegionSampler!!.currentRegionDarkness().isDark + } } else { val isLightTheme = TypedValue() context.theme.resolveAttribute(android.R.attr.isLightTheme, isLightTheme, true) diff --git a/packages/SystemUI/src/com/android/systemui/flags/Flags.kt b/packages/SystemUI/src/com/android/systemui/flags/Flags.kt index 7558b4179a0f..06fc168f95a0 100644 --- a/packages/SystemUI/src/com/android/systemui/flags/Flags.kt +++ b/packages/SystemUI/src/com/android/systemui/flags/Flags.kt @@ -252,7 +252,7 @@ object Flags { // 801 - region sampling // TODO(b/254512848): Tracking Bug - val REGION_SAMPLING = unreleasedFlag(801, "region_sampling") + val REGION_SAMPLING = unreleasedFlag(801, "region_sampling", teamfood = true) // 802 - wallpaper rendering // TODO(b/254512923): Tracking Bug diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/lockscreen/LockscreenSmartspaceController.kt b/packages/SystemUI/src/com/android/systemui/statusbar/lockscreen/LockscreenSmartspaceController.kt index c6911b1ce410..2cf007402410 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/lockscreen/LockscreenSmartspaceController.kt +++ b/packages/SystemUI/src/com/android/systemui/statusbar/lockscreen/LockscreenSmartspaceController.kt @@ -17,6 +17,7 @@ package com.android.systemui.statusbar.lockscreen import android.app.PendingIntent +import android.app.WallpaperManager import android.app.smartspace.SmartspaceConfig import android.app.smartspace.SmartspaceManager import android.app.smartspace.SmartspaceSession @@ -382,7 +383,8 @@ class LockscreenSmartspaceController @Inject constructor( } private fun updateTextColorFromWallpaper() { - if (!regionSamplingEnabled) { + val wallpaperManager = WallpaperManager.getInstance(context) + if (!regionSamplingEnabled || wallpaperManager.lockScreenWallpaperExists()) { val wallpaperTextColor = Utils.getColorAttrDefaultColor(context, R.attr.wallpaperTextColor) smartspaceViews.forEach { it.setPrimaryTextColor(wallpaperTextColor) } -- cgit v1.2.3-59-g8ed1b