[Bug 726193] waylandsink: subsurface & scaling support, plus many other improvements

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=726193
  GStreamer | gst-plugins-bad | git

--- Comment #14 from George Kiagiadakis <george.kiagiadakis at collabora.com> 2014-07-04 16:27:17 UTC ---
(In reply to comment #11) 
> New API and new libraries should be explicitly posted in bugzilla for review,
> and I understand in this case there have even been conversations that indicated
> that this might not be the best or final solution yet.

Well, the purpose of this bug report was exactly that, although I admit I
probably did not make it very clear. Sorry for the confusion.

> Can we disable the new lib for the time being and aim to sort this out in the
> next cycle?

We can, but then waylandsink can't be embedded anywhere. Unless we can disable
it by default and have people build it on demand if needed.


(In reply to comment #13)
> IMHO we should not install the headers of the lib at all as it's not even clear
> that this is how toolkits are supposed to work with Wayland. This is all very
> experimental and not a finished design yet.

About the GstWaylandVideo interface, OK. Maybe we can follow Julien's advice
(bug 732280 comment 3) and transform those two functions into action signals
and kill the interface then.

But what about the GstContext methods that this library contains?

(from the header)
-----------------
/* The type of GstContext used to pass the wl_display pointer
 * from the application to the sink */
#define GST_WAYLAND_DISPLAY_HANDLE_CONTEXT_TYPE
"GstWaylandDisplayHandleContextType"

gboolean gst_is_wayland_display_handle_need_context_message (GstMessage * msg);
GstContext *
gst_wayland_display_handle_context_new (struct wl_display * display);
struct wl_display *
gst_wayland_display_handle_context_get_handle (GstContext * context);
------------------

This should for sure be useful as well to any element that connects to wayland
(ex. glimagesink) and I don't think it should go away.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.