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

Commit 290c8462 authored by Nicolo' Mazzucato's avatar Nicolo' Mazzucato
Browse files

Use DesktopExperienceFlags for keyguard presentation refactor

this enables the refactor when the dev option is on

Bug: 415295016
Test: KeyguardDisplayManagerTest
Flag: com.android.systemui.enable_constraint_layout_lockscreen_on_external_display
Change-Id: Ic4b0fe135dacf09530cdcec9bfa0734e54fd3e47
parent 2534980b
Loading
Loading
Loading
Loading
+58 −0
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.keyguard

import android.window.DesktopExperienceFlags
import com.android.systemui.Flags
import com.android.systemui.flags.RefactorFlagUtils
import com.android.systemui.shade.shared.flag.ShadeWindowGoesAround

/** Helper for reading or using the keyguard presentation refactor class. */
@Suppress("NOTHING_TO_INLINE")
object KeyguardPresentationCLRefactor {

    private const val FLAG_NAME = Flags.FLAG_ENABLE_CONSTRAINT_LAYOUT_LOCKSCREEN_ON_EXTERNAL_DISPLAY

    val FLAG =
        DesktopExperienceFlags.DesktopExperienceFlag(
            Flags::enableConstraintLayoutLockscreenOnExternalDisplay,
            /* shouldOverrideByDevOption= */ true,
            FLAG_NAME,
        )

    /** Is the refactor enabled */
    @JvmStatic
    inline val isEnabled: Boolean
        get() = FLAG.isTrue

    /**
     * Called to ensure code is only run when the flag is enabled. This protects users from the
     * unintended behaviors caused by accidentally running new logic, while also crashing on an eng
     * build to ensure that the refactor author catches issues in testing.
     */
    @JvmStatic
    inline fun isUnexpectedlyInLegacyMode() =
        RefactorFlagUtils.isUnexpectedlyInLegacyMode(isEnabled, ShadeWindowGoesAround.FLAG_NAME)

    /**
     * Called to ensure code is only run when the flag is disabled. This will throw an exception if
     * the flag is enabled to ensure that the refactor author catches issues in testing.
     */
    @JvmStatic
    inline fun assertInLegacyMode() =
        RefactorFlagUtils.assertInLegacyMode(isEnabled, ShadeWindowGoesAround.FLAG_NAME)
}
+2 −2
Original line number Diff line number Diff line
@@ -20,8 +20,8 @@ import android.view.Display
import com.android.keyguard.ConnectedDisplayConstraintLayoutKeyguardPresentation
import com.android.keyguard.ConnectedDisplayKeyguardPresentation
import com.android.keyguard.ConnectedDisplayKeyguardPresentationFactory
import com.android.systemui.Flags
import com.android.systemui.dagger.SysUISingleton
import com.android.systemui.keyguard.KeyguardPresentationCLRefactor
import dagger.Module
import dagger.Provides

@@ -36,7 +36,7 @@ object KeyguardConnectedDisplaysModule {
            ConnectedDisplayConstraintLayoutKeyguardPresentation.Factory,
    ): ConnectedDisplayKeyguardPresentationFactory {
        return ConnectedDisplayKeyguardPresentationFactory { display: Display ->
            if (Flags.enableConstraintLayoutLockscreenOnExternalDisplay()) {
            if (KeyguardPresentationCLRefactor.isEnabled) {
                connectedDisplayConstraintLayoutKeyguardPresentationFactory.create(display)
            } else {
                connectedDisplayKeyguardPresentationFactory.create(display)