[Webkit-unassigned] [Bug 103771] [GStreamer] support in-band text tracks

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=103771





--- Comment #27 from Brendan Long <b.long at cablelabs.com>  2013-04-25 08:15:20 PST ---
(In reply to comment #23)
> What you describe here is exactly what should be inside playbin and exposed via some new API on playbin (and the second item is actually implemented already, and not only for video but audio and text too).

The thing that makes me want to not use playbin is that we need to handle every stream differently, and that seems to not mesh well with using playbin. For video, we want an inputselector, which playbin provides, but for audio we want an adder, and for text we just want all of the pads exposed. It seems like playbin is a one-size-fits-all solution, and that's not really what we need.

> playbin is using uridecodebin and playsink internally, and lots of glue to make them both work perfectly together. The latter is what would have to be reimplemented if you stop using playbin.

I haven't got to video yet, but don't I just need to take any pad I see in pad-added and add it to the video inputselector? The glue I have so far is just handling padAdded() and connecting audio and video to a playsink, and connecting text to a queue then appsink. Ignoring callbacks and other things that I would need anyway, it's only about 40 lines of code, and using uridecodebin makes other parts of the code simpler, since pad-added and pad-removed are easier to work with than text-changed.

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