Integrated: 8268152: gstmpegaudioparse does not provides timestamps for HLS MP3 streams

Sat Jun 5 00:39:57 UTC 2021

On Thu, 3 Jun 2021 09:37:18 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:

> With recent GStreamer update gstmpegaudioparse no longer provides audio buffers with valid timestamps. This issue is only reproducible with MP3 HTTP Live Stream and was noticed on Linux. I think Windows audio decoder handles such case better and thus MP3 HLS works fine. I did not figure out why it behaves this way and for now fix is reverting gstmpegaudioparse changeset which caused it. Follow up issue will be filed to investigate why javasource and/or hlsprogressbuffer no longer compatible with gstmpegaudioparse and gstmpegaudioparse cannot figure out correct PTS. Also, fixed potential null pointer reference for buffers with invalid PTS.

This pull request has now been integrated.

Changeset: ee032387
Author:    Alexander Matveev <almatvee at openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/ee032387badadb41ed36de745aea3c0a074b79bd
Stats:     10 lines in 2 files changed: 10 ins; 0 del; 0 mod

8268152: gstmpegaudioparse does not provides timestamps for HLS MP3 streams

Reviewed-by: kcr, arapte

-------------

PR: https://git.openjdk.java.net/jfx/pull/526