dmaax/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

LibWeb: Move closing of incremental media streams to the fetch cancel

Browse source 
This should ensure that the close() happens in all cases where the
fetch completion may not occur otherwise.
This commit is contained in:
Zaggy1024 2026-04-06 19:24:49 -05:00 committed by Alexander Kalenik
parent 8208d6f3c5
commit a19c40aac5
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
Libraries/LibWeb/HTML/HTMLMediaElement.cpp
View file

@ -262,6 +262,8 @@ void HTMLMediaElement::removed_from(DOM::Node* old_parent, DOM::Node& old_root)
void HTMLMediaElement::cancel_the_fetching_process()
{
m_current_fetch_generation++;
if (m_remote_fetch_data && m_remote_fetch_data->stream)
m_remote_fetch_data->stream->close();
m_remote_fetch_data.clear();
}
@ -1172,11 +1174,7 @@ void HTMLMediaElement::load_url_resource(URL::URL const& url_record, Function<vo
m_remote_fetch_data->stream->set_data_request_callback(GC::weak_callback(*this, [&fetch_data = *m_remote_fetch_data](auto& self, u64 offset) {
self.restart_fetch_at_offset(offset);
}));
m_remote_fetch_data->failure_callback = [&stream = *m_remote_fetch_data->stream, failure_callback = move(failure_callback)](String error_message) {
// Ensure that we unblock any reads if we stop the fetch due to some failure.
stream.close();
failure_callback(move(error_message));
};
m_remote_fetch_data->failure_callback = move(failure_callback);
set_up_playback_manager_for_remote();