[Bug 733369] New: dashdemux fails to detect end of period

[GStreamer (bugzilla.gnome.org)]

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

           Summary: dashdemux fails to detect end of period
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: david.waring at rd.bbc.co.uk
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


Created an attachment (id=281133)
 View: https://bugzilla.gnome.org/attachment.cgi?id=281133
 Review: https://bugzilla.gnome.org/review?bug=733369&attachment=281133

Patch to fix segment start times and period end detection.

dashdemux fails to detect the end of period when the current period doesn't
start at 0s. This seems to be because the period start offset is not added to
the calculated segment start in gst_mpdparser_get_chunk_by_index(), but is
later compared to the end of period offset into the stream.

The attached patch fixes this issue and also a related issue of not progressing
in live streams with multiple periods (should only wait for an MPD update if we
have finished the last period).

I am now seeing another issue which is that the video stops at the end of the
period, it looks like the downloaders for the new period aren't running, but I
still need to track that down. Anyone have an idea why the video downloads are
ceasing?

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