On Fri Jun 2 15:38:27 2023 +0000, Alexandros Frantzis wrote:
`wl_surface` for child windows are not currently used, but will be when we want to enable rendering GL/Vulkan content into child windows. Although the existence of `wl_surface` doesn't really hurt at the moment (except in terms of unused wayland objects, but perhaps I am missing something?), I am fine explicitly not creating them for now, until the accelerated rendering use case becomes a reality. Note that regardless of the existence or not of a `wl_surface`, the current code will composite all child windows into the parent as you describe (i.e., we always return a NULL `window_surface` for them).
Yeah I think it's best to wait for accelerated surface before introducing them, and then only do it on a as-needed basis. Fwiw there can sometimes be a huge number of child Win32 windows created, none of which will need any acceleration, and you probably don't want to create host objects needlessly.