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 695324 - Built-in Recorder Freezes mouse while entering overview
Built-in Recorder Freezes mouse while entering overview
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: st
3.7.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-03-06 20:46 UTC by alex diavatis
Modified: 2013-03-11 17:26 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Mouse Pointer freezes (1.33 MB, video/webm)
2013-03-06 20:46 UTC, alex diavatis 		  Details
recorder: don't bother selecting for enter/leave/motion events (2.30 KB, patch)
2013-03-11 16:35 UTC, Ray Strode [halfline] 		committed Details | Review
recorder: port to XInput2 (10.62 KB, patch)
2013-03-11 16:35 UTC, Ray Strode [halfline] 		committed Details | Review

Description alex diavatis 2013-03-06 20:46:27 UTC 
Created attachment 238244 [details]
Mouse Pointer freezes 

Reproduce always in GS > 3.7.x 
(both with nVidia and Nouveau) 

Pressing Ctrl+Alt+Shift+R. gstreamer will hang up recording mouse movement if we enter overview.

I am attaching a screencasting. Check on the pointer. 

I was trying lots of pipelines, if there is a workaround please let me know as I am trying to make a video.
Comment 1 Ray Strode [halfline] 2013-03-11 16:31:40 UTC 
seems like it just needs to be ported to use XInput2.
Comment 2 Ray Strode [halfline] 2013-03-11 16:35:42 UTC 
Created attachment 238601 [details] [review]
recorder: don't bother selecting for enter/leave/motion events

Mutter already handles that for us.
Comment 3 Ray Strode [halfline] 2013-03-11 16:35:45 UTC 
Created attachment 238602 [details] [review]
recorder: port to XInput2

Pointer tracking is broken when the pointer is over the stage input
area.  This is apparently fallout from mutter going to XInput2.

This commit changes the mouse event handling code to also use XInput2.
Comment 4 Ray Strode [halfline] 2013-03-11 16:36:31 UTC 
The above two patches were only very briefly tested, but they should be good enough that you can get your videos done, I think.
Comment 5 Jasper St. Pierre (not reading bugmail) 2013-03-11 17:01:26 UTC 
Review of attachment 238602 [details] [review]:

OK.
Comment 6 Jasper St. Pierre (not reading bugmail) 2013-03-11 17:01:50 UTC 
Review of attachment 238601 [details] [review]:

OK.
Comment 7 Ray Strode [halfline] 2013-03-11 17:09:22 UTC 
Review of attachment 238602 [details] [review]:

::: src/shell-recorder.c
@@ +732,3 @@
+      if (leave_event->deviceid == VIRTUAL_CORE_POINTER_ID &&
+          (xev->xcrossing.detail != NotifyVirtual &&
+           xev->xcrossing.detail != NotifyNonlinearVirtual))

This part wasn't updated properly.
Comment 8 Ray Strode [halfline] 2013-03-11 17:26:44 UTC 
(pushed with the above mentioned fix)
Attachment 238601 [details] pushed as e80795b - recorder: don't bother selecting for enter/leave/motion events
Attachment 238602 [details] pushed as ddced79 - recorder: port to XInput2