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

LibWeb: Handle language attributes without a '-'

Browse source 
Where we would previously index out of bounds.
This commit is contained in:
Shannon Booth 2024-11-02 03:48:19 +13:00 committed by Jelle Raaijmakers
parent e915143593
commit f20822e293
Notes: github-actions[bot] 2024-11-01 15:10:07 +00:00 
Author: https://github.com/shannonbooth
Commit: https://github.com/LadybirdBrowser/ladybird/commit/f20822e2933
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/2104
Reviewed-by: https://github.com/gmta ✅
7 changed files with 158 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
Tests/LibWeb/Text/expected/wpt-import/html/dom/elements/global-attributes/the-lang-attribute-008.txt Normal file
View file

@ -0,0 +1,11 @@
Summary
Harness status: OK
Rerun
Found 1 tests
1 Pass
Details
Result Test Name MessagePass If an element contains a lang attribute with an empty value, the value of a lang attribute higher up the document tree will no longer be applied to the content of that element.

11
Tests/LibWeb/Text/expected/wpt-import/html/dom/elements/global-attributes/the-lang-attribute-009.txt Normal file
View file

@ -0,0 +1,11 @@
Summary
Harness status: OK
Rerun
Found 1 tests
1 Pass
Details
Result Test Name MessagePass If the HTTP header contains a language declaration but the html element uses an empty lang value, the UA will not recognize the language declared in the HTTP header.

11
Tests/LibWeb/Text/expected/wpt-import/html/dom/elements/global-attributes/the-lang-attribute-010.txt Normal file
View file

@ -0,0 +1,11 @@
Summary
Harness status: OK
Rerun
Found 1 tests
1 Pass
Details
Result Test Name MessagePass If the meta Content-Language element contains a language declaration but the html element uses an empty lang value, the UA will not recognize the language declared in the meta Content-Language element.

41
Tests/LibWeb/Text/input/wpt-import/html/dom/elements/global-attributes/the-lang-attribute-008.html Normal file
View file

@ -0,0 +1,41 @@
<!DOCTYPE html>
<html >
<head>
<meta charset="utf-8"/>
<title>lang="..." vs lang=""</title>
<link rel='author' title='Richard Ishida' href='mailto:ishida@w3.org'>
<link rel='help' href='https://html.spec.whatwg.org/multipage/#the-lang-and-xml:lang-attributes'>
<script src="../../../../resources/testharness.js"></script>
<script src="../../../../resources/testharnessreport.js"></script>
<meta name='flags' content='dom'>
<style type='text/css'>
#colonlangcontroltest { color: red; font-weight: bold; width: 400px; }
#colonlangcontroltest:lang(xx) { display:none; }
.test div { width: 50px; }
#box:lang(ko) { width: 100px; }
</style>
</head>
<body>
<div class="test" lang="ko"><div id="box" lang="">&#xA0;</div></div>
<p lang='xx' id='colonlangcontroltest'>This test failed because it relies on :lang for results, but :lang is not supported by this browser.</p>
<!--Notes:
This test uses :lang to detect whether the language has been set. If :lang is not supported, a message will appear and the test will fail.
-->
<script>
test(function() {
assert_equals(document.getElementById('colonlangcontroltest').offsetWidth, 0)
assert_equals(document.getElementById('box').offsetWidth, 50);
}, "If an element contains a lang attribute with an empty value, the value of a lang attribute higher up the document tree will no longer be applied to the content of that element.");
</script>
<div id='log'></div>
</body>
</html>

41
Tests/LibWeb/Text/input/wpt-import/html/dom/elements/global-attributes/the-lang-attribute-009.html Normal file
View file

@ -0,0 +1,41 @@
<!DOCTYPE html>
<html lang="" >
<head>
<meta charset="utf-8"/>
<title>lang="" vs HTTP</title>
<link rel='author' title='Richard Ishida' href='mailto:ishida@w3.org'>
<link rel='help' href='https://html.spec.whatwg.org/multipage/#the-lang-and-xml:lang-attributes'>
<script src="../../../../resources/testharness.js"></script>
<script src="../../../../resources/testharnessreport.js"></script>
<meta name='flags' content='http dom'>
<style type='text/css'>
#colonlangcontroltest { color: red; font-weight: bold; width: 400px; }
#colonlangcontroltest:lang(xx) { display:none; }
.test div { width: 50px; }
#box:lang(ko) { width: 100px; }
</style>
</head>
<body>
<div class="test"><div id="box">&#xA0;</div></div>
<p lang='xx' id='colonlangcontroltest'>This test failed because it relies on :lang for results, but :lang is not supported by this browser.</p>
<!--Notes:
This test uses :lang to detect whether the language has been set. If :lang is not supported, a message will appear and the test will fail.
-->
<script>
test(function() {
assert_equals(document.getElementById('colonlangcontroltest').offsetWidth, 0)
assert_equals(document.getElementById('box').offsetWidth, 50);
}, "If the HTTP header contains a language declaration but the html element uses an empty lang value, the UA will not recognize the language declared in the HTTP header.");
</script>
<div id='log'></div>
</body>
</html>

42
Tests/LibWeb/Text/input/wpt-import/html/dom/elements/global-attributes/the-lang-attribute-010.html Normal file
View file

@ -0,0 +1,42 @@
<!DOCTYPE html>
<html lang="" >
<head>
<meta charset="utf-8"/>
<meta http-equiv="Content-Language" content="ko" >
<title>lang="" vs meta Content-Language</title>
<link rel='author' title='Richard Ishida' href='mailto:ishida@w3.org'>
<link rel='help' href='https://html.spec.whatwg.org/multipage/#the-lang-and-xml:lang-attributes'>
<script src="../../../../resources/testharness.js"></script>
<script src="../../../../resources/testharnessreport.js"></script>
<meta name='flags' content='dom'>
<style type='text/css'>
#colonlangcontroltest { color: red; font-weight: bold; width: 400px; }
#colonlangcontroltest:lang(xx) { display:none; }
.test div { width: 50px; }
#box:lang(ko) { width: 100px; }
</style>
</head>
<body>
<div class="test"><div id="box">&#xA0;</div></div>
<p lang='xx' id='colonlangcontroltest'>This test failed because it relies on :lang for results, but :lang is not supported by this browser.</p>
<!--Notes:
This test uses :lang to detect whether the language has been set. If :lang is not supported, a message will appear and the test will fail.
-->
<script>
test(function() {
assert_equals(document.getElementById('colonlangcontroltest').offsetWidth, 0)
assert_equals(document.getElementById('box').offsetWidth, 50);
}, "If the meta Content-Language element contains a language declaration but the html element uses an empty lang value, the UA will not recognize the language declared in the meta Content-Language element.");
</script>
<div id='log'></div>
</body>
</html>

2
Userland/Libraries/LibWeb/CSS/SelectorEngine.cpp
View file

@ -72,7 +72,7 @@ static inline bool matches_lang_pseudo_class(DOM::Element const& element, Vector
if (!element_language.contains('-') && Infra::is_ascii_case_insensitive_match(element_language, language)) if (!element_language.contains('-') && Infra::is_ascii_case_insensitive_match(element_language, language))
return true; return true;
auto parts = element_language.split_limit('-', 2).release_value_but_fixme_should_propagate_errors(); auto parts = element_language.split_limit('-', 2).release_value_but_fixme_should_propagate_errors();
if (Infra::is_ascii_case_insensitive_match(parts[0], language)) if (!parts.is_empty() && Infra::is_ascii_case_insensitive_match(parts[0], language))
return true; return true;
} }
return false; return false;