[Bug 707530] qtdemux: Handle segments correctly in push mode seeks

[GStreamer (bugzilla.gnome.org)]

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

Thiago Sousa Santos <thiago.sousa.santos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #23 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2013-09-13 12:27:39 UTC ---
(In reply to comment #22)
> (From update of attachment 254793 [details])
> start_type could be GST_SEEK_TYPE_NONE too btw

True, but I don't think it is properly handled anywhere in demuxers in push
mode. I'm currently improving our gst-validate scenarios to test seeking
further and check if those flags are correctly handled. For now, I've pushed
the fix to this regression. Later I'll run gst-validate to find the other
issues and fix them, if found.

commit 566b0dce403288faf81e653705587c010ba760ca
Author: Thiago Santos <thiago.sousa.santos at collabora.com>
Date:   Thu Sep 12 13:31:01 2013 -0300

    qtdemux: only update stop position if seek requests it

    Check for GST_SEEK_TYPE_NONE for stop poistion and only update
    the stop time if it is requested. Otherwise just maintain whatever
    was stored at the segment

    https://bugzilla.gnome.org/show_bug.cgi?id=707530

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