In XWaylandSurface::map() the reply function, cookie, is never called

[carbonXIII]

mir/src/server/frontend_xwayland/xwayland_surface.cpp

Lines 462 to 487 in 4178136

	// _NET_WM_STATE is not in property_handlers because we only read it on window creation
	// We, the server (not the client) are responsible for updating it after the window has been mapped
	// The client should use a client message to change state later
	auto const cookie = connection->read_property(
	window,
	connection->_NET_WM_STATE,
	{
	[&](std::vector<xcb_atom_t> const& net_wm_states)
	{
	for (auto const& net_wm_state : net_wm_states)
	{
	state = state.with_net_wm_state_change(*connection, NetWmStateAction::ADD, net_wm_state);
	}
	}
	});
	// If we had more properties to read we would queue them all up before completing the first one
	uint32_t const workspace = 1;
	connection->set_property<XCBType::CARDINAL32>(
	window,
	connection->_NET_WM_DESKTOP,
	workspace);
	inform_client_of_window_state(std::unique_lock(mutex), state);
	request_scene_surface_state(state.active_mir_state());
	xcb_map_window(*connection, window);

cookie is not actually called, so properties set by Xwayland clients via _NET_WM_STATE are ignored.