adaptivedemux: Support period change in live playlist

Regardless of LIVE or VOD, "a manifest has next period but
currently EOSed" state is meaning that it's time to advance period.

Previous behavior of adpativedemux, however, was able to period
advancing only for VOD case, since the adaptivedemux tried to
update and wait new manifest without respecting existence of the next period.

https://bugzilla.gnome.org/show_bug.cgi?id=781183
This commit is contained in:
Seungha Yang 2018-05-10 23:08:10 +09:00 committed by Edward Hervey
parent 67ae35813b
commit a8cb7662d7

View file

@ -3783,8 +3783,11 @@ gst_adaptive_demux_stream_download_loop (GstAdaptiveDemuxStream * stream)
if (!demux_class->requires_periodical_playlist_update (demux)) {
ret = gst_adaptive_demux_update_manifest (demux);
break;
} else if (gst_adaptive_demux_stream_wait_manifest_update (demux,
stream)) {
/* Wait only if we can ensure current manifest has been expired.
* The meaning "we have next period" *WITH* EOS is that, current
* period has been ended but we can continue to the next period */
} else if (!gst_adaptive_demux_has_next_period (demux) &&
gst_adaptive_demux_stream_wait_manifest_update (demux, stream)) {
goto end;
}
gst_task_stop (stream->download_task);
@ -3793,6 +3796,8 @@ gst_adaptive_demux_stream_download_loop (GstAdaptiveDemuxStream * stream)
}
} else {
gst_task_stop (stream->download_task);
}
if (gst_adaptive_demux_combine_flows (demux) == GST_FLOW_EOS) {
if (gst_adaptive_demux_has_next_period (demux)) {
GST_DEBUG_OBJECT (stream->pad,
@ -3801,7 +3806,6 @@ gst_adaptive_demux_stream_download_loop (GstAdaptiveDemuxStream * stream)
ret = GST_FLOW_OK;
}
}
}
break;
case GST_FLOW_NOT_LINKED: