[Xfce4-commits] r25831 - in xfwm4/branches/xfce_4_4: . src
Olivier Fourdan
olivier at xfce.org
Fri Jun 22 00:28:04 CEST 2007
Author: olivier
Date: 2007-06-21 22:28:04 +0000 (Thu, 21 Jun 2007)
New Revision: 25831
Modified:
xfwm4/branches/xfce_4_4/NEWS
xfwm4/branches/xfce_4_4/src/client.c
xfwm4/branches/xfce_4_4/src/display.c
xfwm4/branches/xfce_4_4/src/display.h
xfwm4/branches/xfce_4_4/src/hints.c
xfwm4/branches/xfce_4_4/src/hints.h
Log:
Fix xfwm4 hanging with gtk+-2.11.x (Bug #3346) - Patch based on a similar fix for Metacity, see gnome bug #354213
Modified: xfwm4/branches/xfce_4_4/NEWS
===================================================================
--- xfwm4/branches/xfce_4_4/NEWS 2007-06-21 22:21:09 UTC (rev 25830)
+++ xfwm4/branches/xfce_4_4/NEWS 2007-06-21 22:28:04 UTC (rev 25831)
@@ -1,6 +1,7 @@
4.4.2
=====
+- Fix xfwm4 hanging with gtk+-2.11.x (Bug #3346)
- Fix dialogs and modals without parents not being automatically
centered like before (#Bug #3278)
- Fix modifier mask not working with all keynaps (Bug #3194)
Modified: xfwm4/branches/xfce_4_4/src/client.c
===================================================================
--- xfwm4/branches/xfce_4_4/src/client.c 2007-06-21 22:21:09 UTC (rev 25830)
+++ xfwm4/branches/xfce_4_4/src/client.c 2007-06-21 22:28:04 UTC (rev 25831)
@@ -4868,21 +4868,19 @@
if (passdata.c)
{
- GdkPixbuf *icon;
-
TRACE ("entering cycle loop");
passdata.wireframe = wireframeCreate (passdata.c);
- icon = getAppIcon (display_info, passdata.c->window, 32, 32);
passdata.tabwin = tabwinCreate (passdata.c->screen_info->gscr, c,
passdata.c, passdata.cycle_range,
screen_info->params->cycle_workspaces);
eventFilterPush (display_info->xfilter, clientCycleEventFilter, &passdata);
gtk_main ();
eventFilterPop (display_info->xfilter);
- wireframeDelete (screen_info, passdata.wireframe);
TRACE ("leaving cycle loop");
tabwinDestroy (passdata.tabwin);
g_free (passdata.tabwin);
+ wireframeDelete (screen_info, passdata.wireframe);
+ updateXserverTime (display_info);
}
if (passdata.c)
Modified: xfwm4/branches/xfce_4_4/src/display.c
===================================================================
--- xfwm4/branches/xfce_4_4/src/display.c 2007-06-21 22:21:09 UTC (rev 25830)
+++ xfwm4/branches/xfce_4_4/src/display.c 2007-06-21 22:28:04 UTC (rev 25831)
@@ -178,6 +178,19 @@
FALSE, display_info->atoms) != 0);
}
+static void
+myDisplayCreateTimestampWin (DisplayInfo *display_info)
+{
+ XSetWindowAttributes attributes;
+
+ attributes.event_mask = PropertyChangeMask;
+ attributes.override_redirect = TRUE;
+ display_info->timestamp_win =
+ XCreateWindow (display_info->dpy, DefaultRootWindow (display_info->dpy),
+ -100, -100, 10, 10, 0, 0, CopyFromParent, CopyFromParent,
+ CWEventMask | CWOverrideRedirect, &attributes);
+}
+
DisplayInfo *
myDisplayInit (GdkDisplay *gdisplay)
{
@@ -309,6 +322,8 @@
display->resize_cursor[CORNER_COUNT + SIDE_BOTTOM] =
XCreateFontCursor (display->dpy, XC_bottom_side);
+ myDisplayCreateTimestampWin (display);
+
display->xfilter = NULL;
display->screens = NULL;
display->clients = NULL;
@@ -340,6 +355,8 @@
display->move_cursor = None;
XFreeCursor (display->dpy, display->root_cursor);
display->root_cursor = None;
+ XDestroyWindow (display->dpy, display->timestamp_win);
+ display->timestamp_win = None;
if (display->hostname)
{
Modified: xfwm4/branches/xfce_4_4/src/display.h
===================================================================
--- xfwm4/branches/xfce_4_4/src/display.h 2007-06-21 22:21:09 UTC (rev 25830)
+++ xfwm4/branches/xfce_4_4/src/display.h 2007-06-21 22:28:04 UTC (rev 25831)
@@ -226,6 +226,7 @@
gboolean quit;
gboolean reload;
+ Window timestamp_win;
Cursor busy_cursor;
Cursor move_cursor;
Cursor root_cursor;
Modified: xfwm4/branches/xfce_4_4/src/hints.c
===================================================================
--- xfwm4/branches/xfce_4_4/src/hints.c 2007-06-21 22:21:09 UTC (rev 25830)
+++ xfwm4/branches/xfce_4_4/src/hints.c 2007-06-21 22:28:04 UTC (rev 25831)
@@ -1167,27 +1167,18 @@
return setXAtomManagerOwner(display_info, display_info->atoms[atom_id], root, w);
}
-
-static Bool
-checkPropEvent (Display *display, XEvent *xevent, XPointer arg)
+void
+updateXserverTime (DisplayInfo *display_info)
{
- DisplayInfo *display_info;
ScreenInfo *screen_info;
+ char c = '\0';
- display_info = (DisplayInfo *) arg;
- g_return_val_if_fail (display_info, FALSE);
+ g_return_if_fail (display_info);
- screen_info = myDisplayGetDefaultScreen (display_info);
- g_return_val_if_fail (screen_info, FALSE);
-
- if ((xevent->type == PropertyNotify) &&
- (xevent->xproperty.window == screen_info->xfwm4_win) &&
- (xevent->xproperty.atom == display_info->atoms[XFWM4_TIMESTAMP_PROP]))
- {
- return TRUE;
- }
-
- return FALSE;
+ XChangeProperty (display_info->dpy, display_info->timestamp_win,
+ display_info->atoms[XFWM4_TIMESTAMP_PROP],
+ display_info->atoms[XFWM4_TIMESTAMP_PROP],
+ 8, PropModeReplace, (unsigned char *) &c, 1);
}
Time
@@ -1196,7 +1187,6 @@
ScreenInfo *screen_info;
XEvent xevent;
Time timestamp;
- char c = '\0';
g_return_val_if_fail (display_info, (Time) CurrentTime);
timestamp = myDisplayGetCurrentTime (display_info);
@@ -1206,12 +1196,8 @@
g_return_val_if_fail (screen_info, (Time) CurrentTime);
TRACE ("getXServerTime: Using X server roundtrip");
- XChangeProperty (display_info->dpy, screen_info->xfwm4_win,
- display_info->atoms[XFWM4_TIMESTAMP_PROP],
- display_info->atoms[XFWM4_TIMESTAMP_PROP],
- 8, PropModeReplace, (unsigned char *) &c, 1);
- XIfEvent (display_info->dpy, &xevent, checkPropEvent, (XPointer) display_info);
-
+ updateXserverTime (display_info);
+ XWindowEvent (display_info->dpy, display_info->timestamp_win, PropertyChangeMask, &xevent);
timestamp = (Time) myDisplayUpdateCurrentTime (display_info, &xevent);
}
Modified: xfwm4/branches/xfce_4_4/src/hints.h
===================================================================
--- xfwm4/branches/xfce_4_4/src/hints.h 2007-06-21 22:21:09 UTC (rev 25830)
+++ xfwm4/branches/xfce_4_4/src/hints.h 2007-06-21 22:28:04 UTC (rev 25831)
@@ -269,6 +269,7 @@
int,
Window ,
Window);
+void updateXserverTime (DisplayInfo *);
Time getXServerTime (DisplayInfo *);
#ifdef ENABLE_KDE_SYSTRAY_PROXY
More information about the Xfce4-commits
mailing list