283172 2024-11-15 06:05:06 -0800 [Media] Avoid play() call during seek flow before the finishSeek() call 2025-02-13 09:31:35 -0800 1 1 1 Unclassified WebKit Media WebKit Nightly Build Unspecified Unspecified RESOLVED INVALID InRadar P2 Normal --- 285568 1 eocanha eocanha jean-yves.avenard webkit-bug-importer oldest_to_newest 2075348 0 eocanha 2024-11-15 06:05:06 -0800 During the video playback states should ideally flow as play -> pause -> seek_start -> seek_done -> play, but randomly it is observed that the play event is being sent before finishing the seek (seek_done) i.e, play -> pause -> seek_start -> play -> seek_done. Ideally play call might be triggered continuously by the webapp when we are playing a video, so in our scenario between seek_start and seek_done we should avoid the play call. See: https://github.com/WebPlatformForEmbedded/WPEWebKit/pull/1423 2075349 1 eocanha 2024-11-15 06:14:54 -0800 Pull request: https://github.com/WebKit/WebKit/pull/36695 2076997 2 webkit-bug-importer 2024-11-22 06:06:14 -0800 <rdar://problem/140414465> 2078774 3 ews-feeder 2024-12-03 04:14:55 -0800 Committed 287274@main (71a7e0a09291): <https://commits.webkit.org/287274@main> Reviewed commits have been landed. Closing PR #36695 and removing active labels. 2085651 4 jean-yves.avenard 2025-01-07 17:57:16 -0800 This change is wrong. as written in https://github.com/WebPlatformForEmbedded/WPEWebKit/pull/1423 It breaks the spec. There's nothing in the spec that makes the play event being conditional of a seek having been completed. https://html.spec.whatwg.org/multipage/media.html#internal-play-steps if you call play() (even during a seek) and playback is allowed, the play event should be fired. Found be investigating why Duolingo was now broken in Safari STP 210. 2085656 5 jean-yves.avenard 2025-01-07 18:17:05 -0800 Reopened Bugzilla. Broke duolingo, calling play() during a seek() is allowed by the spec. And the play event should be fired even if the seek hasn't completed., tracking revert in https://bugs.webkit.org/show_bug.cgi?id=285568. 2094476 6 eocanha 2025-02-13 09:31:35 -0800 After analyzing the spec and discussing it with the original https://github.com/WebPlatformForEmbedded/WPEWebKit/pull/1423 reporter, we've concluded that the revert is fine and this bug should be resolved as invalid, since it's against the spec. Thanks for having taken the effort to prove your point, Jean-Yves.