[Bug 701509] dashdemux selects first fragment in the manifest for live streams

[GStreamer (bugzilla.gnome.org)]

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

Greg Rutz <greg> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #246522|none                        |needs-work
             status|                            |

--- Comment #5 from Greg Rutz <greg at gsr-tek.com> 2013-06-12 20:37:55 UTC ---
Review of attachment 246522:
 --> (https://bugzilla.gnome.org/review?bug=701509&attachment=246522)

::: ext/dash/gstmpdparser.c
@@ +3259,3 @@
+  seg_duration = gst_mpd_client_get_next_fragment_duration (client);
+  g_assert (seg_duration > 0);
+gst_mpd_client_get_segment_index_at_time (GstMpdClient * client,

The earliest segment start time also needs to take into account the Period
start time.  The Period start time is relative to the MPD
availabilityStartTime, so the two would be added together.  Period start time
is optional, so it will not always be added in.  Also, the optional
MPD at suggestedPresentationDelay should be taken into account and added to the
start time.

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