The last sample of the audio track in the asset used in this test player has a tiny duration (100 ns):
So small, we are truncating it to zero. We're not supposed to have frames with zero duration an. Instead, lets set a minimum frame duration for those fringe cases.
Created attachment 351251 [details]
Comment on attachment 351251 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=351251&action=review
> + m_duration = createMediaTime(std::max(GST_BUFFER_DURATION(buffer), minimumDuration));
I'd recommend moving this above to a more prominent position than inside this method.
Created attachment 351270 [details]
Comment on attachment 351270 [details]
Clearing flags on attachment: 351270
Committed r236679: <https://trac.webkit.org/changeset/236679>
All reviewed patches have been landed. Closing bug.