moving all messages about properties to the top, as a start at #549886

2008-08-30  Thomas Thurman  <tthurman@gnome.org>

        * src/core/window.c (process_property_notify): moving all
	messages about properties to the top, as a start at #549886


svn path=/trunk/; revision=3837
This commit is contained in:
Thomas Thurman 2008-08-30 21:37:13 +00:00 committed by Thomas James Alexander Thurman
parent 347cfab7dd
commit 9c82331009
2 changed files with 16 additions and 32 deletions

View File

@ -1,3 +1,8 @@
2008-08-30 Thomas Thurman <tthurman@gnome.org>
* src/core/window.c (process_property_notify): moving all
messages about properties to the top, as a start at #549886
2008-08-18 Thomas Thurman <tthurman@gnome.org> 2008-08-18 Thomas Thurman <tthurman@gnome.org>
* NEWS: fix version number which broke * NEWS: fix version number which broke

View File

@ -5339,17 +5339,24 @@ process_property_notify (MetaWindow *window,
* this if-else chain. * this if-else chain.
*/ */
if (meta_is_verbose ()) /* avoid looking up the name if we don't have to */
{
char *property_name = XGetAtomName (window->display->xdisplay,
event->atom);
meta_verbose ("Property notify on %s for %s\n",
window->desc, property_name);
XFree (property_name);
}
if (event->atom == XA_WM_NAME) if (event->atom == XA_WM_NAME)
{ {
meta_verbose ("Property notify on %s for WM_NAME\n", window->desc);
/* don't bother reloading WM_NAME if using _NET_WM_NAME already */ /* don't bother reloading WM_NAME if using _NET_WM_NAME already */
if (!window->using_net_wm_name) if (!window->using_net_wm_name)
meta_window_reload_property (window, XA_WM_NAME); meta_window_reload_property (window, XA_WM_NAME);
} }
else if (event->atom == window->display->atom__NET_WM_NAME) else if (event->atom == window->display->atom__NET_WM_NAME)
{ {
meta_verbose ("Property notify on %s for NET_WM_NAME\n", window->desc);
meta_window_reload_property (window, window->display->atom__NET_WM_NAME); meta_window_reload_property (window, window->display->atom__NET_WM_NAME);
/* if _NET_WM_NAME was unset, reload WM_NAME */ /* if _NET_WM_NAME was unset, reload WM_NAME */
@ -5358,15 +5365,12 @@ process_property_notify (MetaWindow *window,
} }
else if (event->atom == XA_WM_ICON_NAME) else if (event->atom == XA_WM_ICON_NAME)
{ {
meta_verbose ("Property notify on %s for WM_ICON_NAME\n", window->desc);
/* don't bother reloading WM_ICON_NAME if using _NET_WM_ICON_NAME already */ /* don't bother reloading WM_ICON_NAME if using _NET_WM_ICON_NAME already */
if (!window->using_net_wm_icon_name) if (!window->using_net_wm_icon_name)
meta_window_reload_property (window, XA_WM_ICON_NAME); meta_window_reload_property (window, XA_WM_ICON_NAME);
} }
else if (event->atom == window->display->atom__NET_WM_ICON_NAME) else if (event->atom == window->display->atom__NET_WM_ICON_NAME)
{ {
meta_verbose ("Property notify on %s for NET_WM_ICON_NAME\n", window->desc);
meta_window_reload_property (window, window->display->atom__NET_WM_ICON_NAME); meta_window_reload_property (window, window->display->atom__NET_WM_ICON_NAME);
/* if _NET_WM_ICON_NAME was unset, reload WM_ICON_NAME */ /* if _NET_WM_ICON_NAME was unset, reload WM_ICON_NAME */
@ -5375,8 +5379,6 @@ process_property_notify (MetaWindow *window,
} }
else if (event->atom == XA_WM_NORMAL_HINTS) else if (event->atom == XA_WM_NORMAL_HINTS)
{ {
meta_verbose ("Property notify on %s for WM_NORMAL_HINTS\n", window->desc);
meta_window_reload_property (window, XA_WM_NORMAL_HINTS); meta_window_reload_property (window, XA_WM_NORMAL_HINTS);
/* See if we need to constrain current size */ /* See if we need to constrain current size */
@ -5384,40 +5386,28 @@ process_property_notify (MetaWindow *window,
} }
else if (event->atom == window->display->atom_WM_PROTOCOLS) else if (event->atom == window->display->atom_WM_PROTOCOLS)
{ {
meta_verbose ("Property notify on %s for WM_PROTOCOLS\n", window->desc);
meta_window_reload_property (window, window->display->atom_WM_PROTOCOLS); meta_window_reload_property (window, window->display->atom_WM_PROTOCOLS);
} }
else if (event->atom == XA_WM_HINTS) else if (event->atom == XA_WM_HINTS)
{ {
meta_verbose ("Property notify on %s for WM_HINTS\n", window->desc);
meta_window_reload_property (window, XA_WM_HINTS); meta_window_reload_property (window, XA_WM_HINTS);
} }
else if (event->atom == window->display->atom__MOTIF_WM_HINTS) else if (event->atom == window->display->atom__MOTIF_WM_HINTS)
{ {
meta_verbose ("Property notify on %s for MOTIF_WM_HINTS\n", window->desc);
meta_window_reload_property (window, meta_window_reload_property (window,
window->display->atom__MOTIF_WM_HINTS); window->display->atom__MOTIF_WM_HINTS);
} }
else if (event->atom == XA_WM_CLASS) else if (event->atom == XA_WM_CLASS)
{ {
meta_verbose ("Property notify on %s for WM_CLASS\n", window->desc);
meta_window_reload_property (window, XA_WM_CLASS); meta_window_reload_property (window, XA_WM_CLASS);
} }
else if (event->atom == XA_WM_TRANSIENT_FOR) else if (event->atom == XA_WM_TRANSIENT_FOR)
{ {
meta_verbose ("Property notify on %s for WM_TRANSIENT_FOR\n", window->desc);
meta_window_reload_property (window, XA_WM_TRANSIENT_FOR); meta_window_reload_property (window, XA_WM_TRANSIENT_FOR);
} }
else if (event->atom == else if (event->atom ==
window->display->atom_WM_WINDOW_ROLE) window->display->atom_WM_WINDOW_ROLE)
{ {
meta_verbose ("Property notify on %s for WM_WINDOW_ROLE\n", window->desc);
update_role (window); update_role (window);
} }
else if (event->atom == else if (event->atom ==
@ -5430,12 +5420,10 @@ process_property_notify (MetaWindow *window,
else if (event->atom == else if (event->atom ==
window->display->atom__NET_WM_WINDOW_TYPE) window->display->atom__NET_WM_WINDOW_TYPE)
{ {
meta_verbose ("Property notify on %s for NET_WM_WINDOW_TYPE\n", window->desc);
update_net_wm_type (window); update_net_wm_type (window);
} }
else if (event->atom == window->display->atom__NET_WM_ICON) else if (event->atom == window->display->atom__NET_WM_ICON)
{ {
meta_verbose ("Property notify on %s for NET_WM_ICON\n", window->desc);
meta_icon_cache_property_changed (&window->icon_cache, meta_icon_cache_property_changed (&window->icon_cache,
window->display, window->display,
event->atom); event->atom);
@ -5443,8 +5431,6 @@ process_property_notify (MetaWindow *window,
} }
else if (event->atom == window->display->atom__KWM_WIN_ICON) else if (event->atom == window->display->atom__KWM_WIN_ICON)
{ {
meta_verbose ("Property notify on %s for KWM_WIN_ICON\n", window->desc);
meta_icon_cache_property_changed (&window->icon_cache, meta_icon_cache_property_changed (&window->icon_cache,
window->display, window->display,
event->atom); event->atom);
@ -5453,20 +5439,15 @@ process_property_notify (MetaWindow *window,
else if ((event->atom == window->display->atom__NET_WM_STRUT) || else if ((event->atom == window->display->atom__NET_WM_STRUT) ||
(event->atom == window->display->atom__NET_WM_STRUT_PARTIAL)) (event->atom == window->display->atom__NET_WM_STRUT_PARTIAL))
{ {
meta_verbose ("Property notify on %s for _NET_WM_STRUT\n", window->desc);
meta_window_update_struts (window); meta_window_update_struts (window);
} }
else if (event->atom == window->display->atom__NET_STARTUP_ID) else if (event->atom == window->display->atom__NET_STARTUP_ID)
{ {
meta_verbose ("Property notify on %s for _NET_STARTUP_ID\n", window->desc);
meta_window_reload_property (window, meta_window_reload_property (window,
window->display->atom__NET_STARTUP_ID); window->display->atom__NET_STARTUP_ID);
} }
else if (event->atom == window->display->atom__NET_WM_SYNC_REQUEST_COUNTER) else if (event->atom == window->display->atom__NET_WM_SYNC_REQUEST_COUNTER)
{ {
meta_verbose ("Property notify on %s for _NET_WM_SYNC_REQUEST_COUNTER\n", window->desc);
meta_window_reload_property (window, meta_window_reload_property (window,
window->display->atom__NET_WM_SYNC_REQUEST_COUNTER); window->display->atom__NET_WM_SYNC_REQUEST_COUNTER);
} }
@ -5475,8 +5456,6 @@ process_property_notify (MetaWindow *window,
Window xid; Window xid;
Atom atom__NET_WM_USER_TIME; Atom atom__NET_WM_USER_TIME;
meta_verbose ("Property notify on %s for _NET_WM_USER_TIME\n", window->desc);
atom__NET_WM_USER_TIME = window->display->atom__NET_WM_USER_TIME; atom__NET_WM_USER_TIME = window->display->atom__NET_WM_USER_TIME;
if (window->user_time_window) if (window->user_time_window)
xid = window->user_time_window; xid = window->user_time_window;