285569 2025-01-07 18:27:34 -0800 Add test so that bug 285568 can't happen again 2025-01-08 06:07:33 -0800 1 1 1 Unclassified WebKit Media WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=285595 InRadar P2 Normal --- 1 jean-yves.avenard jean-yves.avenard webkit-bug-importer oldest_to_newest 2085660 0 jean-yves.avenard 2025-01-07 18:27:34 -0800 In https://bugs.webkit.org/show_bug.cgi?id=285568 , we reverted https://bugs.webkit.org/show_bug.cgi?id=283172 as it broke a major web site (Duolingo). The change in 283172 prevented a call to `MediaPlayer::play()` if we were in the middle of a seek (it had exception to bypass the new logic) if currentTime was >= duration which isn't sufficient, currentTime could be at 0 if we do seek to 0 -> play() Having play() being called and acted upon during seek() is allowed by the spec. If GStreamer has an issue there, it should be handled in MediaPlayerPrivateGStreamer rather than in HTMLMediaElement. We need a test so that any future change as done in bug 283172 can't happen again. 2085661 1 webkit-bug-importer 2025-01-07 18:27:55 -0800 <rdar://problem/142517881> 2085728 2 jean-yves.avenard 2025-01-08 01:21:23 -0800 Pull request: https://github.com/WebKit/WebKit/pull/38710 2085773 3 ews-feeder 2025-01-08 06:07:31 -0800 Committed 288590@main (4bdbf870a50a): <https://commits.webkit.org/288590@main> Reviewed commits have been landed. Closing PR #38710 and removing active labels.