beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 23:50:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWeb: Update `shared_declarative_refresh_steps()" to use navigables

Browse source
This commit is contained in:
Aliaksandr Kalenik 2023-08-25 00:09:56 +02:00 committed by Andreas Kling
parent 912cf89727
commit c0c542d495
Notes: sideshowbarker 2024-07-16 22:11:09 +09:00 
Author: https://github.com/kalenikaliaksandr
Commit: https://github.com/SerenityOS/serenity/commit/c0c542d495
Pull-request: https://github.com/SerenityOS/serenity/pull/18219
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibWeb/DOM/Document.cpp
View file

@ -3479,11 +3479,8 @@ void Document::shared_declarative_refresh_steps(StringView input, JS::GCPtr<HTML
if (has_meta_element && has_flag(active_sandboxing_flag_set(), HTML::SandboxingFlagSet::SandboxedAutomaticFeatures))
return;
// FIXME: Use navigables when they're used for all navigation (otherwise, navigable() would be null in some cases)
VERIFY(browsing_context());
auto request = Fetch::Infrastructure::Request::create(vm());
request->set_url(url_record);
MUST(browsing_context()->navigate(request, *browsing_context(), false, HTML::HistoryHandlingBehavior::Replace));
VERIFY(navigable());
MUST(navigable()->navigate(url_record, *this));
}).release_value_but_fixme_should_propagate_errors();
// For the purposes of the previous paragraph, a refresh is said to have come due as soon as the later of the