221854 2021-02-12 15:58:35 -0800 Cues displayed during end time 2022-06-17 16:19:26 -0700 1 1 1 Unclassified WebKit Media Safari 14 Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 joeyparrish webkit-unassigned eric.carlson jer.noble webkit-bug-importer oldest_to_newest 1728829 0 420187 joeyparrish 2021-02-12 15:58:35 -0800 Created attachment 420187 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 1728830 1 joeyparrish 2021-02-12 15:59:32 -0800 Corresponding Chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1178026 1729687 2 eric.carlson 2021-02-16 08:48:44 -0800 Thanks for the report Joey! 1729725 3 joeyparrish 2021-02-16 09:39:19 -0800 Happy to help! Thanks for taking a look. 1731654 4 webkit-bug-importer 2021-02-19 15:59:13 -0800 <rdar://problem/74541188> 1869806 5 y_soliman 2022-05-17 14:32:30 -0700 Pull request: https://github.com/WebKit/WebKit/pull/694 1876552 6 ews-feeder 2022-06-17 16:19:24 -0700 Committed r295647 (251652@main): <https://commits.webkit.org/251652@main> Reviewed commits have been landed. Closing PR #694 and removing active labels. 420187 2021-02-12 15:58:35 -0800 2021-02-12 15:58:35 -0800 A simple page (45 lines) to repro the issue cue-edge-case.html text/html 1525 joeyparrish PGh0bWw+Cjxib2R5PgogIDx2aWRlbyBpZD0idmlkZW8iCiAgICAgIHN0eWxlPSJ3aWR0aDogMTAw JTsgaGVpZ2h0OiAxMDAlOyIKICAgICAgc3JjPSJodHRwczovL3N0b3JhZ2UuZ29vZ2xlYXBpcy5j b20vc2hha2EtZGVtby1hc3NldHMvc2ludGVsLW1wNC1vbmx5L3YtMDE0NHAtMDEwMGstbGlieDI2 NC5tcDQiIC8+CjwvYm9keT4KPHNjcmlwdD4KCihhc3luYyAoKSA9PiB7CiAgY29uc29sZS5sb2co J0luc2VydGluZyBjdWVzLicpOwogIHRleHRUcmFjayA9IHZpZGVvLmFkZFRleHRUcmFjaygnc3Vi dGl0bGVzJyk7CiAgdGV4dFRyYWNrLm1vZGUgPSAnc2hvd2luZyc7CiAgdGV4dFRyYWNrLmFkZEN1 ZShuZXcgVlRUQ3VlKDEsIDIsICdUaGlzIHNob3VsZCBiZSBnb25lIGJ5IDJzLicpKTsKICB0ZXh0 VHJhY2suYWRkQ3VlKG5ldyBWVFRDdWUoMiwgMywgJ1RoaXMgc2hvdWxkIGFwcGVhciBhbG9uZSBh dCAycy4nKSk7CgogIC8vIFNvbWUgYnJvd3NlcnMgZG9uJ3QgZGlzcGxheSBjdWVzIHVudGlsIHRo ZSB2aWRlbyBwbGF5cy4gIFNvIHBsYXkgaXQgYQogIC8vIGxpdHRsZSBmaXJzdC4KICBjb25zb2xl LmxvZygnUGxheWluZy4nKTsKICBhd2FpdCB2aWRlby5wbGF5KCk7CiAgaWYgKHZpZGVvLnJlYWR5 U3RhdGUgPCBIVE1MTWVkaWFFbGVtZW50LkhBVkVfRlVUVVJFX0RBVEEpIHsKICAgIGF3YWl0IG5l dyBQcm9taXNlKChyZXNvbHZlKSA9PiB2aWRlby5hZGRFdmVudExpc3RlbmVyKCdjYW5wbGF5Jywg cmVzb2x2ZSkpOwogIH0KICBhd2FpdCBuZXcgUHJvbWlzZSgocmVzb2x2ZSkgPT4gc2V0VGltZW91 dChyZXNvbHZlLCA1MDApKTsKCiAgLy8gTm93IHBhdXNlIGFuZCBzZWVrIHRvIGV4YWN0bHkgMnMu ICBUaGUgc2Vjb25kIHN1YnRpdGxlIHNob3VsZCBhcHBlYXIgYWxvbmUsCiAgLy8gYmVjYXVzZSB0 aGUgZGlzcGxheSByYW5nZSBvZiBhIGN1ZSBpbmNsdWRlcyB0aGUgc3RhcnQgdGltZSwgYnV0IGV4 Y2x1ZGVzCiAgLy8gdGhlIGVuZCB0aW1lLgogIGNvbnNvbGUubG9nKCdTZWVraW5nLicpOwogIHZp ZGVvLnBhdXNlKCk7CiAgdmlkZW8uY3VycmVudFRpbWUgPSAyOwoKICAvLyBXYWl0IGEgbGl0dGxl IHRvIG1ha2Ugc3VyZSB0aGUgZGlzcGxheSBoYXMgYmVlbiB1cGRhdGVkLgogIGNvbnNvbGUubG9n KCdXYWl0aW5nLicpOwogIGF3YWl0IG5ldyBQcm9taXNlKChyZXNvbHZlKSA9PiBzZXRUaW1lb3V0 KHJlc29sdmUsIDUwMCkpOwoKICAvLyBUaGlzIGVkZ2UgY2FzZSBpcyBoYW5kbGVkIGNvcnJlY3Rs eSBpbiBGaXJlZm94IDg0LCBidXQgaW4gQ2hyb21lIDg4IGFuZAogIC8vIFNhZmFyaSAxNCwgYm90 aCBjdWVzIGFwcGVhciB0b2dldGhlci4KICBpZiAodGV4dFRyYWNrLmFjdGl2ZUN1ZXMgJiYgdGV4 dFRyYWNrLmFjdGl2ZUN1ZXMubGVuZ3RoICE9IDEpIHsKICAgIGNvbnNvbGUuZXJyb3IoCiAgICAg IGBTaG91bGQgZGlzcGxheSBleGFjdGx5IDEgY3VlLCBub3QgJHt0ZXh0VHJhY2suYWN0aXZlQ3Vl cy5sZW5ndGh9IWApOwogIH0KfSkoKTsKCjwvc2NyaXB0Pgo8L2h0bWw+Cg==