After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 564561 - unset _NET_SUPPORTING_WM_CHECK when shutting down
unset _NET_SUPPORTING_WM_CHECK when shutting down
Status: RESOLVED FIXED
Product: metacity
Classification: Other
Component: EWMH specification
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Metacity maintainers list
Metacity maintainers list
: 564562 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-12-15 05:38 UTC by Matthias Clasen
Modified: 2009-07-09 14:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (919 bytes, patch)
2008-12-15 05:39 UTC, Matthias Clasen
committed Details | Review

Description Matthias Clasen 2008-12-15 05:38:47 UTC
This helps to prevent an obscure problem that we've run into recently.
If metacity leaves _NET_SUPPORTING_WM_CHECK behind (a), and another app manages to reuse that window id (b) before another wm has taken over (c), bad things happen. GTK+ apps that are started between (a) and (c) won't know about the window manager started in (c) until the app started in (b) dies, since they keep using the value of _NET_SUPPORTING_WM_CHECK they got initially.

The attached patch unsets _NET_SUPPORTING_WM_CHECK when metacity stops managing a screen. Note that this is only best-effort, since the property will still be left behind when metacity is just killed.
Comment 1 Matthias Clasen 2008-12-15 05:39:20 UTC
Created attachment 124705 [details] [review]
patch
Comment 2 Matthias Clasen 2008-12-15 05:40:04 UTC
https://bugzilla.redhat.com/show_bug.cgi?id=471927 has the full story
Comment 3 Akira TAGOH 2008-12-15 06:14:14 UTC
*** Bug 564562 has been marked as a duplicate of this bug. ***
Comment 4 Thomas Thurman 2009-07-09 14:27:55 UTC
Committed with thanks.