From 1f6466895326defef370a378f9d7b61a7e246df7 Mon Sep 17 00:00:00 2001
From: Tim Waterhouse <tim@timwaterhouse.com>
Date: Sat, 13 Jul 2024 03:53:23 -0700
Subject: [PATCH] ext-foreign-toplevel: Send done after title and class (#6857)

According to the spec (https://wayland.app/protocols/ext-foreign-toplevel-list-v1#ext_foreign_toplevel_handle_v1:event:title), clients should wait for the done signal before applying updates
---
 src/protocols/ForeignToplevel.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/protocols/ForeignToplevel.cpp b/src/protocols/ForeignToplevel.cpp
index 40420da7..f7b3886f 100644
--- a/src/protocols/ForeignToplevel.cpp
+++ b/src/protocols/ForeignToplevel.cpp
@@ -81,6 +81,7 @@ void CForeignToplevelList::onTitle(PHLWINDOW pWindow) {
         return;
 
     H->resource->sendTitle(pWindow->m_szTitle.c_str());
+    H->resource->sendDone();
 }
 
 void CForeignToplevelList::onClass(PHLWINDOW pWindow) {
@@ -92,6 +93,7 @@ void CForeignToplevelList::onClass(PHLWINDOW pWindow) {
         return;
 
     H->resource->sendAppId(pWindow->m_szClass.c_str());
+    H->resource->sendDone();
 }
 
 void CForeignToplevelList::onUnmap(PHLWINDOW pWindow) {