kwin_wayland and setWindowIcon()

[Elvis Angelaccio]

On lunedì 11 settembre 2017 22:15:20 CEST, David Edmundson wrote:
> From a KDE side we can't use that as we don't have it.
>
> There is no icon name part of the specification in either of 
> the shell protocols.
> The only relevant event is AppId. 
>
> Qt could base the appId off the window icon but it wouldn't 
> make any sense; (code for this and it's fallbacks if the 
> .desktopfile are not set are in qwaylandwindow.cpp:163->183)
>
> I very very much doubt the sent data would be any different in weston.
> You can run WAYLAND_DEBUG=1 qtcreator and grep for the icon 
> name if you like. 
>
> If you see no output it means you're running in xwayland, which 
> would also be a reason it works.
>
> The only other thing that might be different is their appId -> 
> desktop file -> icon resolution; 
> You can show find out what Qt sends for QtCreator using that 
> debug line, and then see there's a reason we don't find the 
> relevant .desktop file.

Interesting, weston shows:

    zxdg_toplevel_v6 at 45.set_app_id("qtcreator")

while kwin doesn't. So it looks like they are using the name of the binary 
as fallback?

QWaylandWindow does the same but prepends the organization domain, maybe 
that's where it breaks.

Cheers,
Elvis