[Bug 723674] Make external sink(s) compatible with gst-gl

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #12 from Matthew Waters <ystreet00 at gmail.com> 2014-02-23 06:04:33 UTC ---
(In reply to comment #11)
> Well done for the already 3 commited patchs. For the display part it will be
> also useful for https://bugzilla.gnome.org/show_bug.cgi?id=709747#c10. I will
> try to split
> https://bug709747.bugzilla-attachments.gnome.org/attachment.cgi?id=266903.

:)

> (In reply to comment #9)
> Have a look at the first attached file (external_sink_wrap_gl_callback /
> "share").
> (It assumes one of the clutter gl threads (and I guess there is only one) is
> running in the main GMainContext but we could add this callback in the right
> GMainContext if that was not the case.)

It's not guaranteed that there will only ever be one clutter GMainContext. 
Clutter has expressed interest in going multithreaded although that doesn't
seem to be materialising anytime soon.  Also, there is the policy within
GStreamer that we do not rely on there ever being a GMainContext at any point
in time.

> Upstream gl elements will response to [GST_GL_TYPE_CONTEXT, sink->context,
> "share", G_TYPE_BOOLEAN, TRUE] the same way as for
> https://bugzilla.gnome.org/show_bug.cgi?id=704809. Except here cluttersink
> explicitly tells downstream it has to share.

The problem here is getting the gl context out of cluttersink cleanly [1] as
cogl does not directly support what we are trying to do (yet? :)).  Everything
else should work from our point of view.

[1] There are the {egl,glX}GetCurrentContext() family of functions however we
cannot really know which to call.  I would prefer to define some minimal
integration requirements rather than relying on some heuristics.

-- 
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.