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 387138 - x input events processing in sinks with xoverlay interface [patch]
x input events processing in sinks with xoverlay interface [patch]
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
0.10.x
Other Linux
: Normal enhancement
: 0.10.12
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-12-18 13:03 UTC by Radek Doulik
Modified: 2007-01-04 11:35 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed patch (6.86 KB, patch)
2006-12-18 13:04 UTC, Radek Doulik 		none Details | Review
Another patch (14.28 KB, patch)
2007-01-03 19:55 UTC, Julien MOUTTE 		none Details | Review

Description Radek Doulik 2006-12-18 13:03:44 UTC 
Hi all,

 	while working on gstreamer support for OpenOffice I run into problem
where xvimagesink and ximagesink are eating keypress/keyrelease events
in OOo embedded window (I set xwindow ID with GstXOverlay iface) and I
am unable to receive them in OOo.

	I am attaching a patch which introduces new method
gst_x_overlay_set_process_input in GstXOverlay interface. This method
can be used to tell X based video sinks to not process input events. The
patch also implements this new function in xvimage and ximage sinks.
Comment 1 Radek Doulik 2006-12-18 13:04:31 UTC 
Created attachment 78562 [details] [review]
proposed patch
Comment 2 Julien MOUTTE 2007-01-03 19:55:21 UTC 
Created attachment 79307 [details] [review]
Another patch

That patch provides the support for disabling events on internal and external windows even on the fly.
Comment 3 Stefan Sauer (gstreamer, gtkdoc dev) 2007-01-04 10:17:10 UTC 
+1 from me,

toggle_events (GstXOverlay * ov)
should probably use GST_DEBUG() and not g_print
Comment 4 Julien MOUTTE 2007-01-04 11:35:04 UTC 
stress-overlay is a stress test element it's using g_print everywhere for now. I don't really see the point of switching to debugging macros.

Patch entered CVS. Bug fixed.