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

LibWeb: Append attributes to the correct element

Browse source 
The spec indicates we should append attributes to the top element of the
stack of open elements. We were appending the attribute to the bottom.
This commit is contained in:
Timothy Flynn 2024-07-29 17:22:35 -04:00 committed by Andreas Kling
parent 9fe35ddddf
commit 657bbd1542
Notes: github-actions[bot] 2024-07-30 07:42:32 +00:00 
Author: https://github.com/trflynn89
Commit: https://github.com/LadybirdBrowser/ladybird/commit/657bbd1542c
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/894
3 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Tests/LibWeb/Text/expected/unclosed-html-element.txt
View file

@ -1 +1 @@
PASS (didn't crash)
HTML element has '<head' attribute: true

2
Tests/LibWeb/Text/input/unclosed-html-element.html
View file

@ -5,6 +5,6 @@
<script src="include.js"></script>
<script>
test(() => {
println("PASS (didn't crash)");
println(`HTML element has '<head' attribute: ${document.documentElement.hasAttribute('<head')}`);
});
</script>

5
Userland/Libraries/LibWeb/HTML/Parser/HTMLParser.cpp
View file

@ -1719,9 +1719,10 @@ void HTMLParser::handle_in_body(HTMLToken& token)
// Otherwise, for each attribute on the token, check to see if the attribute is already present on the top element of the stack of open elements.
// If it is not, add the attribute and its corresponding value to that element.
auto& top_element = m_stack_of_open_elements.first();
token.for_each_attribute([&](auto& attribute) {
if (!current_node().has_attribute(attribute.local_name))
current_node().append_attribute(attribute.local_name, attribute.value);
if (!top_element.has_attribute(attribute.local_name))
top_element.append_attribute(attribute.local_name, attribute.value);
return IterationDecision::Continue;
});
return;