RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-10-31 06:32:44 +00:00
Code Issues Activity Actions

LibWeb: Remove exit for javascript urls in anchor activation_behavior()

Browse source 
This early return is no longer needed because Navigable::navigate()
can process `javascript:` urls.

Fixes https://github.com/SerenityOS/serenity/issues/22052
This commit is contained in:
Aliaksandr Kalenik 2023-11-25 13:58:13 +01:00 • committed by Andreas Kling
parent 80d1c93edf
commit afe24883b7
3 changed files with 15 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

1
Tests/LibWeb/Text/expected/anchor-element-with-javascript-url-href.txt Normal file
View file

@ -0,0 +1 @@
link clicked!

14
Tests/LibWeb/Text/input/anchor-element-with-javascript-url-href.html Normal file
View file

@ -0,0 +1,14 @@
<script src="include.js"></script>
<script>
function javascript_navigation_callback() {
println("link clicked!");
window.done();
}
</script>
<a id="test-link" href="javascript:javascript_navigation_callback()"></a>
<script>
asyncTest((done) => {
document.getElementById("test-link").click();
window.done = done;
});
</script>

5
Userland/Libraries/LibWeb/HTML/HTMLAnchorElement.cpp
View file

@ -57,11 +57,6 @@ void HTMLAnchorElement::activation_behavior(Web::DOM::Event const&)
if (href().is_empty()) if (href().is_empty())
return; return;
// AD-HOC: follow_the_hyperlink currently doesn't navigate properly with javascript urls
// EventHandler::handle_mouseup performs the navigation steps for javascript urls instead
if (href().starts_with_bytes("javascript:"sv))
return;
// 2. Let hyperlinkSuffix be null. // 2. Let hyperlinkSuffix be null.
Optional<String> hyperlink_suffix {}; Optional<String> hyperlink_suffix {};