[Bug 734471] asfdemux: Should pass seek events upstream if not supported

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=734471
  GStreamer | gst-plugins-ugly | unspecified

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk
            Summary|[asfdemux] Should pass seek |asfdemux: Should pass seek
                   |events upstream if not      |events upstream if not
                   |supported                   |supported

--- Comment #2 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2014-08-08 10:25:02 UTC ---
Please note that we have been using 'element: xyz' in bug titles for the last
couple of years instead of '[element] xyz'.

I think what you want to do here is right in principle, but I'm not sure if the
patch is right. asfdemux (and any demuxer, at least when operating in push
mode) should already try sending a seek upstream first before attempting to
process it itself. In your patch you do that as a fallback, but it should be
tried first, and I'm sure it does that already for mms and rtsp, so question is
why doesn't it do it for your case? Is your stream marked as non-seekable in
the headers? Perhaps that's something that should be fixed too?

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