RFR: 8259356: MediaPlayer's seek freezes video

Johan Vos jvos at openjdk.java.net
Sat Apr 17 15:51:34 UTC 2021


On Sat, 17 Apr 2021 01:39:17 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:

> This is regression (introduced) by JDK-8199527. JDK-8199527 added fix for HLS streams (unfortunately I was not able to find repro case and more details on why it was added) in gstappsink.c line 659-678 to store current caps which can be lost during seek/flush. However, this workaround broke code in gst_app_sink_render_common() line 939 which was restoring caps if they were flushed, since we set last_caps to non NULL value. Since code in gst_app_sink_render_common() did not work to propagate caps to sample our rendering code was dropping frames without caps. Fixed by setting caps on sample in HLS workaround code.

Marked as reviewed by jvos (Reviewer).

Verified on Linux and MacOS, looks good.

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

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


More information about the openjfx-dev mailing list