RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-23 03:07:39 +00:00
Code Issues Activity Actions

LibWeb: Add AO for navigation_must_be_a_replace to Navigable

Browse source 
This is used by both Navigable and Navigation, so let's put it in
Navigable. Also add a missing AK/String include to make clangd happier
with the Navigable file.
This commit is contained in:
Andrew Kaster 2023-08-24 16:18:40 -06:00 • committed by Andreas Kling
parent 7ad45d9e71
commit 34ea470d4f
2 changed files with 9 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Userland/Libraries/LibWeb/HTML/Navigable.cpp
View file

@ -1113,4 +1113,10 @@ void Navigable::reload()
}); });
} }
// https://html.spec.whatwg.org/multipage/browsing-the-web.html#the-navigation-must-be-a-replace
bool navigation_must_be_a_replace(AK::URL const& url, DOM::Document const& document)
{
return url.scheme() == "javascript"sv || document.is_initial_about_blank();
}
} }

3
Userland/Libraries/LibWeb/HTML/Navigable.h
View file

@ -7,6 +7,7 @@
#pragma once #pragma once
#include <AK/String.h>
#include <LibJS/Heap/Cell.h> #include <LibJS/Heap/Cell.h>
#include <LibWeb/Forward.h> #include <LibWeb/Forward.h>
#include <LibWeb/HTML/ActivateTab.h> #include <LibWeb/HTML/ActivateTab.h>
@ -135,4 +136,6 @@ private:
JS::GCPtr<NavigableContainer> m_container; JS::GCPtr<NavigableContainer> m_container;
}; };
bool navigation_must_be_a_replace(AK::URL const& url, DOM::Document const& document);
} }