[Bug 741322] New: dashdemux: Change period when setting by period id.

[GStreamer (bugzilla.gnome.org)]

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

           Summary: dashdemux: Change period when setting by period id.
    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=292423)
 View: https://bugzilla.gnome.org/attachment.cgi?id=292423
 Review: https://bugzilla.gnome.org/review?bug=741322&attachment=292423

Patch to change period when setting by period id.

At present gst_mpd_client_set_period_id doesn't actually set the current
period, it just returns true if it finds a period with a matching id.

This affects manifest updates where the Period/@id attribute is set in the MPD
and the current playing period is not the first one.

This will cause playback to jump back to the first period on manifest update,
and is demonstrated by:
http://rdmedia.bbc.co.uk/dash/testmpds/multiperiod/bbb.php

Attached is a patch which will make gst_mpd_client_set_period_id set the
current period index when it finds a period with a matching period id.

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