[Bug 734659] qtdemux: Does not properly update base time for non-flushing seeks

[GStreamer (bugzilla.gnome.org)]

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

Aleix Conchillo Flaqué <aleix> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #283388|0                           |1
        is obsolete|                            |

--- Comment #27 from Aleix Conchillo Flaqué <aleix at oblong.com> 2014-09-02 20:56:04 UTC ---
Created an attachment (id=285192)
 View: https://bugzilla.gnome.org/attachment.cgi?id=285192
 Review: https://bugzilla.gnome.org/review?bug=734659&attachment=285192

update segment base time for non-flushing seeks

Thiago, thank you for the review! I think I have addressed the two comments.

I don't know if this is even possible, but I have one concern: if there are
video and audio streams and one stream segment (let's say video) is activated
twice consecutively before the other had the chance to be activated. I think
this patch would fail, because in the second activation the largest position
would be the audio one (as it was not activated the first time).

Let me know if this has any chance to happen and I'll try to think of another
way to fix it.

Thanks!

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