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 412486 - gnome-screensaver doesn't work with xnest
gnome-screensaver doesn't work with xnest
Status: RESOLVED FIXED
Product: gnome-screensaver
Classification: Deprecated
Component: daemon
CVS HEAD
Other All
: Normal normal
: ---
Assigned To: gnome-screensaver maintainers
gnome-screensaver maintainers
Depends on:
Blocks:
 
 
Reported: 2007-02-27 01:17 UTC by William Lachance
Modified: 2007-02-28 20:02 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18

Attachments
Patch to make gnome-screensaver not watch for mouse motion events from windows which haven't registered for such events (1.30 KB, patch)
2007-02-27 01:19 UTC, William Lachance 		committed Details | Review

Description William Lachance 2007-02-27 01:17:34 UTC 
Please describe the problem:
As seen from ubuntu: https://launchpad.net/bugs/84662

From my tracing, it seems like gnome-screensaver is trying to watch for mouse motion events for the Xnest root window (which hasn't registered for such events), preventing Xnest itself from receiving the mouse motion event. Of course, I may gotten this wrong.

Steps to reproduce:
Xnest -ac :20&
DISPLAY=:20 xeyes&

Actual results:
xeyes doesn't follow the cursor reliably.

Expected results:
xeyes should follow the cursor reliably.

Does this happen every time?
Yes.

Other information:
Comment 1 William Lachance 2007-02-27 01:19:15 UTC 
Created attachment 83436 [details] [review]
Patch to make gnome-screensaver not watch for mouse motion events from windows which haven't registered for such events

Patch which should fix the issue, treating mouse motion events similarly to keyboard input events (i.e.: ignore them if the window isn't watching for them). Also took the liberty of making some minor cleanups to the surrounding issue
Comment 2 Jonh Wendell 2007-02-28 12:58:24 UTC 
I have tried this patch and it works fine.
Comment 3 Laurent Bigonville 2007-02-28 16:56:40 UTC 
Yep work fine, thanks :)
Comment 4 William Jon McCann 2007-02-28 20:02:53 UTC 
Nice catch.  Committed.  Thanks!