Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: cddba712f11ce83043909d167c1cf9124c7e9e8f
https://github.com/WebKit/WebKit/commit/cddba712f11ce83043909d167c1cf9124c7e9e8f
Author: Ahmad Saleem <[email protected]>
Date: 2026-09-02 (Wed, 02 Sep 2026)
Changed paths:
M LayoutTests/TestExpectations
M Source/WebCore/mathml/MathMLElement.cpp
Log Message:
-----------
MathML rowspan="0" is incorrectly clamped to 1
https://bugs.webkit.org/show_bug.cgi?id=323176
rdar://186420659
Reviewed by Frédéric Wang Nélar.
MathMLElement::rowSpan() used std::max(1u, ...), which clobbered
rowspan="0" up to 1. Per spec, rowspan="0" means the cell spans all
remaining rows of its row group, and the render layer already handles
this via RenderTableCell::calculateRowSpanForRowSpanZero(). HTML table
cells get this right through clampHTMLNonNegativeIntegerToRange() with a
minimum of 0; MathML's mtd was not mirroring that behavior.
Mirror HTMLTableCellElement::rowSpan() by using
clampHTMLNonNegativeIntegerToRange() with minRowspan (0), maxRowspan,
and defaultRowspan, so a rowspan of 0 reaches the render layer intact.
* LayoutTests/TestExpectations: Unskip now progressed test
* Source/WebCore/mathml/MathMLElement.cpp:
(WebCore::MathMLElement::rowSpan const):
Canonical link: https://commits.webkit.org/320324@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
