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

LibWeb: Layout all math elements using InternalDummy context

Browse source 
Always create a new formatting context for <math> elements. Previously
that didn't happen if they only had inline children, e.g. mtable.

This fixes a crash in the WPT MathML test
mathml/crashtests/children-with-negative-block-sizes.html
This commit is contained in:
David Smith 2024-10-15 18:53:19 +02:00 committed by Alexander Kalenik
parent 4408ea7c9b
commit e7c209820d
Notes: github-actions[bot] 2024-10-16 17:52:28 +00:00 
Author: https://github.com/stendavid
Commit: https://github.com/LadybirdBrowser/ladybird/commit/e7c209820d2
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/1831
3 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
Tests/LibWeb/Text/expected/math-with-inline-child.txt Normal file
View file

@ -0,0 +1 @@
PASS (didn't crash)

11
Tests/LibWeb/Text/input/math-with-inline-child.html Normal file
View file

@ -0,0 +1,11 @@
<!DOCTYPE html>
<math>
<mtable></mtable>
</math>
<script src="include.js"></script>
<script>
test(() => {
println("PASS (didn't crash)");
});
</script>

4
Userland/Libraries/LibWeb/Layout/FormattingContext.cpp
View file

@ -136,6 +136,10 @@ Optional<FormattingContext::Type> FormattingContext::formatting_context_type_cre
if (display.is_grid_inside())
return Type::Grid;
if (display.is_math_inside())
// HACK: Instead of crashing, create a dummy formatting context that does nothing.
return Type::InternalDummy;
if (creates_block_formatting_context(box))
return Type::Block;