frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Allow events to grabWindows event if the device is not grabbed. 

This kinda makes popup windows useable if the WM doesn't set the
ClientPointer. Kinda.
This commit is contained in:
Peter Hutterer 2007-04-27 13:24:27 +09:30
parent cfc01115af
commit 339b73e710
2 changed files with 22 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
dix/events.c
View File

@ -1729,7 +1729,7 @@ DeliverEventsToWindow(DeviceIntPtr pDev, WindowPtr pWin, xEvent
return 0; return 0;
if (!(type & EXTENSION_EVENT_BASE) && if (!(type & EXTENSION_EVENT_BASE) &&
IsInterferingGrab(wClient(pWin), pDev, pEvents)) IsInterferingGrab(wClient(pWin), pWin, pDev, pEvents))
return 0; return 0;
if ( (attempt = TryClientEvents(wClient(pWin), pEvents, count, if ( (attempt = TryClientEvents(wClient(pWin), pEvents, count,
@ -1762,7 +1762,7 @@ DeliverEventsToWindow(DeviceIntPtr pDev, WindowPtr pWin, xEvent
{ {
/* core event? check for grab interference */ /* core event? check for grab interference */
if (!(type & EXTENSION_EVENT_BASE) && if (!(type & EXTENSION_EVENT_BASE) &&
IsInterferingGrab(rClient(other), pDev, pEvents)) IsInterferingGrab(rClient(other), pWin, pDev, pEvents))
continue; continue;
if ( (attempt = TryClientEvents(rClient(other), pEvents, count, if ( (attempt = TryClientEvents(rClient(other), pEvents, count,
@ -5097,15 +5097,19 @@ PickKeyboard(ClientPtr client)
/* A client that has one or more core grabs does not get core events from /* A client that has one or more core grabs does not get core events from
* devices it does not have a grab on. Legacy applications behave bad * devices it does not have a grab on. Legacy applications behave bad
* otherwise because they are not used to it and the events interfere. * otherwise because they are not used to it and the events interfere.
* The one exception is: if we're about to send an event to a window that is
* specified as grab window, we still do it. This makes popup menus
* half-useable for WMs that don't set the ClientPointer.
* Only applies for core events. * Only applies for core events.
* *
* Return true if a core event from the device would interfere and should not * Return true if a core event from the device would interfere and should not
* be delivered. * be delivered.
*/ */
Bool Bool
IsInterferingGrab(ClientPtr client, DeviceIntPtr dev, xEvent* event) IsInterferingGrab(ClientPtr client, WindowPtr win, DeviceIntPtr dev, xEvent* event)
{ {
DeviceIntPtr it = inputInfo.devices; DeviceIntPtr it;
Bool mayInterfere = FALSE;
if (dev->coreGrab.grab && SameClient(dev->coreGrab.grab, client)) if (dev->coreGrab.grab && SameClient(dev->coreGrab.grab, client))
return FALSE; return FALSE;
@ -5124,19 +5128,30 @@ IsInterferingGrab(ClientPtr client, DeviceIntPtr dev, xEvent* event)
return FALSE; return FALSE;
} }
it = inputInfo.devices;
while(it) while(it)
{ {
if (it != dev) if (it != dev)
{ {
if (it->coreGrab.grab && SameClient(it->coreGrab.grab, client)) if (it->coreGrab.grab && SameClient(it->coreGrab.grab, client))
{ {
return TRUE; /* there's a client with a grab on some device.
* if we're delivering to the very same window that is
* grabbed (or a child), we're good */
WindowPtr parent = win;
while(parent)
{
if (it->coreGrab.grab->window == parent)
return FALSE;
parent = parent->parent;
}
mayInterfere = TRUE;
} }
} }
it = it->next; it = it->next;
} }
return FALSE; return mayInterfere;
} }

1
include/dix.h
View File

@ -550,6 +550,7 @@ extern DeviceIntPtr PickKeyboard(
extern Bool IsInterferingGrab( extern Bool IsInterferingGrab(
ClientPtr /* client */, ClientPtr /* client */,
WindowPtr /* win */,
DeviceIntPtr /* dev */, DeviceIntPtr /* dev */,
xEvent* /* events */); xEvent* /* events */);