From 4531717f3ee58f6093f62e5afb21da6599b5a813 Mon Sep 17 00:00:00 2001
From: vaxerski <43317083+vaxerski@users.noreply.github.com>
Date: Wed, 4 Oct 2023 13:39:40 +0100
Subject: [PATCH] renderer: considerable fixes to repaint logic

---
 src/events/Monitors.cpp        | 3 ++-
 src/helpers/SubsurfaceTree.cpp | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/events/Monitors.cpp b/src/events/Monitors.cpp
index 0b90ba1e..82577dae 100644
--- a/src/events/Monitors.cpp
+++ b/src/events/Monitors.cpp
@@ -156,7 +156,8 @@ void Events::listener_monitorFrame(void* owner, void* data) {
         if (!PMONITOR->tearingState.frameScheduledWhileBusy)
             return; // we did not schedule a frame yet to be displayed, but we are tearing. Why render?
 
-        PMONITOR->tearingState.nextRenderTorn = true;
+        PMONITOR->tearingState.nextRenderTorn          = true;
+        PMONITOR->tearingState.frameScheduledWhileBusy = false;
     }
 
     static auto* const PENABLERAT = &g_pConfigManager->getConfigValuePtr("misc:render_ahead_of_time")->intValue;
diff --git a/src/helpers/SubsurfaceTree.cpp b/src/helpers/SubsurfaceTree.cpp
index 76a392e8..b0ed0bfa 100644
--- a/src/helpers/SubsurfaceTree.cpp
+++ b/src/helpers/SubsurfaceTree.cpp
@@ -249,7 +249,8 @@ void Events::listener_commitSubsurface(void* owner, void* data) {
     if (pNode->pWindowOwner) {
         // tearing: if solitary, redraw it. This still might be a single surface window
         const auto PMONITOR = g_pCompositor->getMonitorFromID(pNode->pWindowOwner->m_iMonitorID);
-        if (PMONITOR->solitaryClient == pNode->pWindowOwner && pNode->pWindowOwner->canBeTorn() && PMONITOR->tearingState.canTear) {
+        if (PMONITOR->solitaryClient == pNode->pWindowOwner && pNode->pWindowOwner->canBeTorn() && PMONITOR->tearingState.canTear &&
+            pNode->pSurface->wlr()->current.committed & WLR_SURFACE_STATE_BUFFER) {
 
             CRegion damageBox;
             wlr_surface_get_effective_damage(pNode->pSurface->wlr(), damageBox.pixman());