tint2 and _NET_WM_ICON

[Olivier Fourdan]

On Fri, Sep 18, 2009 at 11:26 AM, Olivier Fourdan <fourdan at gmail.com> wrote:
[...]

> But thre request has no chance to be accepted, NET_WM_ICON has to be
> set by the apps, not the window manager. Doing otherwise is 1) not
> logic and 2) would introduce a race condition with the application
> (whch is supposed to set the property on its window) - So fix the app,
> do not break the window manager.

Note that in that case the app that needs fixing in the application
itself, not "tint2".

I am really surprised that openbox does that, because it clearly
introduces a race condition between the window manager and the
application itself.

1) the app does not set an icon (for example some apps may change
their icon depending on their status)
2) The WM forces a default icon by setting the property NET_WM_ICON itself
3) The app and the window manager may end up setting the same property
at the same time => race cond.

Cheers,
Olivier.