Bug 270645
| Summary: | [Cocoa] WebCodecs HEVC decoder is not always reordering frames according presentation time | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Marcello Bastea-Forte <marcello> |
| Component: | Media | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit-bug-importer, youennf |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Mac (Apple Silicon) | ||
| OS: | macOS 14 | ||
Marcello Bastea-Forte
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 | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
youenn fablet
Should be fixed after https://commits.webkit.org/275434@main
*** This bug has been marked as a duplicate of bug 270096 ***
Marcello Bastea-Forte
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
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
(on macOS 14.3.1 (23D60))
Radar WebKit Bug Importer
<rdar://problem/127655559>
youenn fablet
I can reproduce the issue, our VPS reading routine might need to cover more cases.
EWS
Committed 278462@main (1fb6c8c6fa42): <https://commits.webkit.org/278462@main>
Reviewed commits have been landed. Closing PR #28231 and removing active labels.