'xdg_menu' check was redundant, 'is_closed' is not

as the loading of menus can take a very long time and the connection may be closed by the time it has finished

xpra/server/mixins/child_command.py

@@ -165,8 +165,9 @@ def send_xdg_menu_data(self, ss) -> None:
         if ss.is_closed():
             return
         xdg_menu = self._get_xdg_menu_data() or {}
-        if ss.xdg_menu:
-            ss.send_setting_change("xdg-menu", xdg_menu)
+        if ss.is_closed():
+            return
+        ss.send_setting_change("xdg-menu", xdg_menu)
         log(f"{len(xdg_menu)} menu data entries sent to {ss}")
 
     def send_updated_menu(self, xdg_menu) -> None: