OSSForks/ladybird
1
0
Fork 0
mirror of https://github.com/zebrajr/ladybird.git synced 2025-12-06 12:20:00 +01:00
Code Issues Actions 3 Packages Projects Releases Wiki Activity

LibWeb: Don't loop the media element when paused

Browse Source 
We should be able to seek to the end of the media without looping if
we're paused.

See: https://github.com/whatwg/html/issues/11774
This commit is contained in:
Zaggy1024 2025-10-10 15:36:13 -05:00 committed by Jelle Raaijmakers
parent 4471e8c0ec
commit cb1719aa81
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Libraries/LibWeb/HTML/HTMLMediaElement.cpp
View File

@ -1626,6 +1626,13 @@ WebIDL::ExceptionOr<void> HTMLMediaElement::play_element()
// notify about playing for the element. // notify about playing for the element.
else { else {
notify_about_playing(); notify_about_playing();
// AD-HOC: If the official playback position is at the end of the media data here, that means that we haven't
// run the reached the end of media playback steps. This can happen if we seeked to the end while paused
// and looping.
// See https://github.com/whatwg/html/issues/11774
if (m_official_playback_position == m_duration)
reached_end_of_media_playback();
} }
} }
@ -2033,7 +2040,10 @@ void HTMLMediaElement::reached_end_of_media_playback()
m_loop_was_specified_when_reaching_end_of_media_resource = has_attribute(HTML::AttributeNames::loop); m_loop_was_specified_when_reaching_end_of_media_resource = has_attribute(HTML::AttributeNames::loop);
if (m_loop_was_specified_when_reaching_end_of_media_resource) { if (m_loop_was_specified_when_reaching_end_of_media_resource) {
// then seek to the earliest possible position of the media resource and return. // then seek to the earliest possible position of the media resource and return.
seek_element(0); // AD-HOC: We don't want to loop back to the start if we're paused.
// See https://github.com/whatwg/html/issues/11774
if (!paused())
seek_element(0);
// FIXME: Tell PlaybackManager that we're looping to allow data providers to decode frames ahead when looping // FIXME: Tell PlaybackManager that we're looping to allow data providers to decode frames ahead when looping
// and remove any delay in displaying the first frame again. // and remove any delay in displaying the first frame again.
return; return;