From f5149524e8232571d7929c9ac7708c10a8b51cc9 Mon Sep 17 00:00:00 2001
From: vaxerski <43317083+vaxerski@users.noreply.github.com>
Date: Sun, 24 Sep 2023 17:12:15 +0100
Subject: [PATCH] move unsafe checks to frame

---
 src/config/ConfigManager.cpp | 3 ---
 src/events/Monitors.cpp      | 6 +++++-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/config/ConfigManager.cpp b/src/config/ConfigManager.cpp
index eb5edde1..8c83824f 100644
--- a/src/config/ConfigManager.cpp
+++ b/src/config/ConfigManager.cpp
@@ -2020,9 +2020,6 @@ void CConfigManager::performMonitorReload() {
     if (overAgain)
         performMonitorReload();
 
-    if (!g_pCompositor->m_vMonitors.empty()) // reset unsafe state if we have monitors
-        g_pCompositor->leaveUnsafeState();
-
     m_bWantsMonitorReload = false;
 
     EMIT_HOOK_EVENT("monitorLayoutChanged", nullptr);
diff --git a/src/events/Monitors.cpp b/src/events/Monitors.cpp
index 9d17fd34..947f9e7f 100644
--- a/src/events/Monitors.cpp
+++ b/src/events/Monitors.cpp
@@ -112,7 +112,6 @@ void Events::listener_newOutput(wl_listener* listener, void* data) {
         }
 
         g_pCompositor->m_bReadyToProcess = true;
-        g_pCompositor->leaveUnsafeState();
     }
 
     g_pConfigManager->m_bWantsMonitorReload = true;
@@ -139,6 +138,11 @@ void Events::listener_monitorFrame(void* owner, void* data) {
     if ((g_pCompositor->m_sWLRSession && !g_pCompositor->m_sWLRSession->active) || !g_pCompositor->m_bSessionActive || g_pCompositor->m_bUnsafeState) {
         Debug::log(WARN, "Attempted to render frame on inactive session!");
 
+        if (g_pCompositor->m_bUnsafeState && PMONITOR->output != g_pCompositor->m_pUnsafeOutput) {
+            // restore from unsafe state
+            g_pCompositor->leaveUnsafeState();
+        }
+
         return; // cannot draw on session inactive (different tty)
     }