From 164b0fb268035576a4bac8a1e5c17900469f68f6 Mon Sep 17 00:00:00 2001 From: Alejandro Nijamkin Date: Mon, 17 Oct 2022 12:31:47 -0700 Subject: [PATCH] Turns off use of refactord UserSwitcherController. There was an automatically detected massive regression in performance, as seen in the attached bug. This flag change shuts down the use of UserInteractor instead of the controller. Now, we have time to locally investigate the performance regression before turning that on again. Fix: 253435919 Test: verified that user switcher is still working Change-Id: I716439dcbe55e998687229ab0e68176548eff685 --- packages/SystemUI/src/com/android/systemui/flags/Flags.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/flags/Flags.java b/packages/SystemUI/src/com/android/systemui/flags/Flags.java index 5c3a9b4ebe245..0832ae461add7 100644 --- a/packages/SystemUI/src/com/android/systemui/flags/Flags.java +++ b/packages/SystemUI/src/com/android/systemui/flags/Flags.java @@ -111,8 +111,8 @@ public class Flags { *

If this is {@code false}, the interactor and repo skip the controller and directly access * the framework APIs. */ - public static final UnreleasedFlag USER_INTERACTOR_AND_REPO_USE_CONTROLLER = - new UnreleasedFlag(210); + public static final ReleasedFlag USER_INTERACTOR_AND_REPO_USE_CONTROLLER = + new ReleasedFlag(210); /** * Whether `UserSwitcherController` should use the user interactor. @@ -123,7 +123,7 @@ public class Flags { *

Note: do not set this to true if {@link #USER_INTERACTOR_AND_REPO_USE_CONTROLLER} is * {@code true} as it would created a cycle between controller -> interactor -> controller. */ - public static final ReleasedFlag USER_CONTROLLER_USES_INTERACTOR = new ReleasedFlag(211); + public static final UnreleasedFlag USER_CONTROLLER_USES_INTERACTOR = new UnreleasedFlag(211); /***************************************/ // 300 - power menu