Bug 221854 - Cues displayed during end time
Summary: Cues displayed during end time
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Safari 14
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-02-12 15:58 PST by Joey Parrish
Modified: 2022-06-17 16:19 PDT (History)
3 users (show)

See Also:


Attachments
A simple page (45 lines) to repro the issue (1.49 KB, text/html)
2021-02-12 15:58 PST, Joey Parrish
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Joey Parrish 2021-02-12 15:58:35 PST
Created attachment 420187 [details]
A simple page (45 lines) to repro the issue

Safari version 14

What steps will reproduce the problem?
(1) Open the attached HTML repro case (45 lines total)
(2) Optional: observe the sequence of events in the dev console logs

What is the expected result?

The repro page will insert two cues (1-2s and 2-3s), then seek to exactly 2s.
Only one cue should be displayed at time 2s (the second one).

What happens instead?

In Chrome/Edge 88 and Safari 14, both are displayed at once.  In Firefox 84, only the second cue is displayed (correct behavior).


Additional context:

It recently was pointed out to us in Shaka Player that our DOM-UI-based cue display got this wrong, too.  The fix turned out to be a single character (> instead of >=).  You may find something similar in WebKit.  https://github.com/google/shaka-player/pull/3152/files
Comment 1 Joey Parrish 2021-02-12 15:59:32 PST
Corresponding Chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1178026
Comment 2 Eric Carlson 2021-02-16 08:48:44 PST
Thanks for the report Joey!
Comment 3 Joey Parrish 2021-02-16 09:39:19 PST
Happy to help!  Thanks for taking a look.
Comment 4 Radar WebKit Bug Importer 2021-02-19 15:59:13 PST
<rdar://problem/74541188>
Comment 5 Youssef Soliman 2022-05-17 14:32:30 PDT
Pull request: https://github.com/WebKit/WebKit/pull/694
Comment 6 EWS 2022-06-17 16:19:24 PDT
Committed r295647 (251652@main): <https://commits.webkit.org/251652@main>

Reviewed commits have been landed. Closing PR #694 and removing active labels.