protocols/cm: Fix preferred image description (#11026)

src/protocols/ColorManagement.cpp

@@ -410,14 +410,27 @@ CColorManagementFeedbackSurface::CColorManagementFeedbackSurface(SP<CWpColorMana
         m_currentPreferred = RESOURCE;
 
         m_currentPreferred->m_settings = m_surface->getPreferredImageDescription();
-        if (!PROTO::colorManagement->m_debug && m_currentPreferred->m_settings.icc.fd) {
+        m_currentPreferredId           = m_currentPreferred->m_settings.updateId();
+
+        if (!PROTO::colorManagement->m_debug && m_currentPreferred->m_settings.icc.fd >= 0) {
             LOGM(ERR, "FIXME: parse icc profile");
             r->error(WP_COLOR_MANAGER_V1_ERROR_UNSUPPORTED_FEATURE, "ICC profiles are not supported");
             return;
         }
 
-        RESOURCE->resource()->sendReady(m_currentPreferred->m_settings.updateId());
+        RESOURCE->resource()->sendReady(m_currentPreferredId);
     });
+
+    m_listeners.enter = m_surface->m_events.enter.listen([this](const auto& monitor) { onPreferredChanged(); });
+    m_listeners.leave = m_surface->m_events.leave.listen([this](const auto& monitor) { onPreferredChanged(); });
+}
+
+void CColorManagementFeedbackSurface::onPreferredChanged() {
+    if (m_surface->m_enteredOutputs.size() == 1) {
+        const auto newId = m_surface->getPreferredImageDescription().updateId();
+        if (m_currentPreferredId != newId)
+            m_resource->sendPreferredChanged(newId);
+    }
 }
 
 bool CColorManagementFeedbackSurface::good() {
@@ -796,6 +809,9 @@ void CColorManagementProtocol::onMonitorImageDescriptionChanged(WP<CMonitor> mon
         if (output->m_monitor == monitor)
             output->m_resource->sendImageDescriptionChanged();
     }
+    // recheck feedbacks
+    for (auto const& feedback : m_feedbackSurfaces)
+        feedback->onPreferredChanged();
 }
 
 void CColorManagementProtocol::destroyResource(CColorManager* resource) {

src/protocols/ColorManagement.hpp

@@ -89,6 +89,14 @@ class CColorManagementFeedbackSurface {
     wl_client*                              m_client = nullptr;
 
     WP<CColorManagementImageDescription>    m_currentPreferred;
+    uint32_t                                m_currentPreferredId = 0;
+
+    struct {
+        CHyprSignalListener enter;
+        CHyprSignalListener leave;
+    } m_listeners;
+
+    void onPreferredChanged();
 
     friend class CColorManagementProtocol;
 };

src/protocols/core/Compositor.cpp

@@ -307,6 +307,7 @@ void CWLSurfaceResource::enter(PHLMONITOR monitor) {
     m_enteredOutputs.emplace_back(monitor);
 
     m_resource->sendEnter(output->getResource().get());
+    m_events.enter.emit(monitor);
 }
 
 void CWLSurfaceResource::leave(PHLMONITOR monitor) {
@@ -323,6 +324,7 @@ void CWLSurfaceResource::leave(PHLMONITOR monitor) {
     std::erase(m_enteredOutputs, monitor);
 
     m_resource->sendLeave(output->getResource().get());
+    m_events.leave.emit(monitor);
 }
 
 void CWLSurfaceResource::sendPreferredTransform(wl_output_transform t) {
@@ -539,7 +541,18 @@ void CWLSurfaceResource::commitState(SSurfaceState& state) {
 }
 
 SImageDescription CWLSurfaceResource::getPreferredImageDescription() {
-    return m_enteredOutputs.size() == 1 ? m_enteredOutputs[0]->m_imageDescription : g_pCompositor->getPreferredImageDescription();
+    auto parent = m_self;
+    if (parent->m_role->role() == SURFACE_ROLE_SUBSURFACE) {
+        auto subsurface = ((CSubsurfaceRole*)parent->m_role.get())->m_subsurface.lock();
+        parent          = subsurface->t1Parent();
+    }
+    WP<CMonitor> monitor;
+    if (parent->m_enteredOutputs.size() == 1)
+        monitor = parent->m_enteredOutputs[0];
+    else if (m_hlSurface.valid() && m_hlSurface->getWindow())
+        monitor = m_hlSurface->getWindow()->m_monitor;
+
+    return monitor ? monitor->m_imageDescription : g_pCompositor->getPreferredImageDescription();
 }
 
 void CWLSurfaceResource::updateCursorShm(CRegion damage) {

src/protocols/core/Compositor.hpp

@@ -87,6 +87,8 @@ class CWLSurfaceResource {
         CSignalT<>                          unmap;
         CSignalT<SP<CWLSubsurfaceResource>> newSubsurface;
         CSignalT<>                          destroy;
+        CSignalT<SP<CMonitor>>              enter;
+        CSignalT<SP<CMonitor>>              leave;
     } m_events;
 
     SSurfaceState                          m_current;