commit a0858db67d1e45da995cf23b24e23ca83846b7d0
author Dominik Laskowski <domlaskowski@google.com> Tue Jul 23 16:45:11 2024 -0400
committer Dominik Laskowski <domlaskowski@google.com> Thu Jul 25 15:23:22 2024 -0400
tree 477e9a402d83723d68bb203aa36614572cae73ba
parent 8805c3ea813e0dd9f537ced0cf84ee60ced8fb91

SF: Remove fallback for outdated active display

Initialize mActiveDisplayId during boot. getDefaultDisplay's fallback is
then no longer needed, since mActiveDisplayId is an internal display, so
cannot be removed and become outdated.

Bug: 255635821
Flag: EXEMPT refactor
Test: Boot
Change-Id: Ifa1a1f679957e88788b1819ba117fda3f407430b

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 22a8993..216fb43 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -911,9 +911,11 @@
     LOG_ALWAYS_FATAL_IF(!configureLocked(),
                         "Initial display configuration failed: HWC did not hotplug");
 
+    mActiveDisplayId = getPrimaryDisplayIdLocked();
+
     // Commit primary display.
     sp<const DisplayDevice> display;
-    if (const auto indexOpt = mCurrentState.getDisplayIndex(getPrimaryDisplayIdLocked())) {
+    if (const auto indexOpt = mCurrentState.getDisplayIndex(mActiveDisplayId)) {
         const auto& displays = mCurrentState.displays;
 
         const auto& token = displays.keyAt(*indexOpt);