RESOLVED FIXED270645
[Cocoa] WebCodecs HEVC decoder is not always reordering frames according presentation time
https://bugs.webkit.org/show_bug.cgi?id=270645
Summary [Cocoa] WebCodecs HEVC decoder is not always reordering frames according pres...
Marcello Bastea-Forte
Reported 2024-03-07 11:23:11 PST
Related: https://bugs.webkit.org/show_bug.cgi?id=268987, maybe followup for https://bugs.webkit.org/show_bug.cgi?id=265389 ? The h265/HEVC test file on the webcodecs sample is returning decoded frames out of order: Full repro steps: - pull the code: - either from https://github.com/marcello3d/webcodecs/tree/marcello/frame-out-of-order-logging - apply the patch yourself to https://github.com/w3c/webcodecs https://patch-diff.githubusercontent.com/raw/w3c/webcodecs/pull/734.patch - run the server with `node server` in the samples folder - open http://localhost:8888/ in safari - open developer console to see javascript logs - Leave renderer at 2D and select H.265 video codec - click start Result: - 4 frames decoded then, Error: Decoded frame 166666 µs out of order, last frame was 200000 µs Expected: - it decodes the whole file
Attachments
youenn fablet
Comment 1 2024-03-07 11:28:33 PST
Should be fixed after https://commits.webkit.org/275434@main *** This bug has been marked as a duplicate of bug 270096 ***
Marcello Bastea-Forte
Comment 2 2024-03-07 13:22:04 PST
awesome, I guess that missed the STP 190 cutoff, I'll test in 191! Do you have a sense of when this fix would make it to end-users?
Marcello Bastea-Forte
Comment 3 2024-04-15 11:08:14 PDT
This is still broken for me (same error from test in first comment) in Safari Technology Preview Release 192 (Safari 17.4, WebKit 19619.1.8.1)
Marcello Bastea-Forte
Comment 4 2024-04-15 11:08:48 PDT
(on macOS 14.3.1 (23D60))
Radar WebKit Bug Importer
Comment 5 2024-05-07 02:12:13 PDT
youenn fablet
Comment 6 2024-05-07 02:13:01 PDT
I can reproduce the issue, our VPS reading routine might need to cover more cases.
EWS
Comment 7 2024-05-07 09:18:39 PDT
Committed 278462@main (1fb6c8c6fa42): <https://commits.webkit.org/278462@main> Reviewed commits have been landed. Closing PR #28231 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.