Bug 180013 – WPT tests for MathML lengths fail

• Component: MathML
• Status: NEW

[Frédéric Wang (:fredw)]

* http://www.w3c-test.org/mathml/relations/css-styling/lengths-1.html
* http://www.w3c-test.org/mathml/relations/css-styling/lengths-3.html
  The "unitless" tests fail because mathsize="5" and mathsize="20.0" is not interpreted as "500%" and "2000%". See https://www.w3.org/TR/MathML/chapter2.html#fund.units

* http://www.w3c-test.org/mathml/relations/css-styling/lengths-2.html
  This test fails because of limitation the mpadded element ( https://bugs.webkit.org/show_bug.cgi?id=85730 ), probably the lack of support for pseudo units and for negative lengths. See https://www.w3.org/TR/MathML/chapter3.html#presm.mpadded

[Rob Buis]

Created attachment 362701 [details]
Patch

[Frédéric Wang (:fredw)]

Comment on attachment 362701 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=362701&action=review

> Source/WebCore/ChangeLog:3
> +        WPT tests for MathML lengths fail

This actually fixes the mathsize unitless and tests length-1 and length-3. I guess it could be in a different bug with better title/desc. At least we should have a bug open to track the remaining length-2 failure.

(Note: I wonder if the support for (nonzero) unitless values should be removed from MathML)

> Source/WebCore/ChangeLog:8
> +        Covert unitless lengths to percentage values to correct the computed

convert

[Rob Buis]

(In reply to Frédéric Wang (:fredw) from comment #2)
> Comment on attachment 362701 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=362701&action=review
> 
> > Source/WebCore/ChangeLog:3
> > +        WPT tests for MathML lengths fail
> 
> This actually fixes the mathsize unitless and tests length-1 and length-3. I
> guess it could be in a different bug with better title/desc. At least we
> should have a bug open to track the remaining length-2 failure.

Sure, I opened https://bugs.webkit.org/show_bug.cgi?id=194940 instead and removed the review flag for this one.

> (Note: I wonder if the support for (nonzero) unitless values should be
> removed from MathML)
> 
> > Source/WebCore/ChangeLog:8
> > +        Covert unitless lengths to percentage values to correct the computed
> 
> convert

Fixed.

[Rob Buis]

Not working on this ATM, but only failure left:

* http://www.w3c-test.org/mathml/relations/css-styling/lengths-2.html
  This test fails because of limitation the mpadded element ( https://bugs.webkit.org/show_bug.cgi?id=85730 ), probably the lack of support for pseudo units and for negative lengths. See https://www.w3.org/TR/MathML/chapter3.html#presm.mpadded