namedspaces are not supported in mpadded attributes

RESOLVED INVALID

Status

()

Core
MathML
RESOLVED INVALID
2 years ago
2 years ago

People

(Reporter: fredw, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox49 affected)

Details

(URL)

(Reporter)

Description

2 years ago
That seems to be a bug in our implementation...

Testcase: http://tests.mathml-association.org/mathml/relations/css-styling/lengths-2.html
The testcase falls over here

https://hg.mozilla.org/mozilla-central/annotate/16663eb3dcfa759f25b5e27b101bc79270c156f2/layout/mathml/nsMathMLmpaddedFrame.cpp#l172

The namedspaces are used in the format voffset="thickmathspace", yet nsMathMLmpaddedFrame requires at least one digit to be present, claiming that this is to follow the specs.

I've looked up the spec and this seems to be correct.

https://www.w3.org/TR/MathML/appendixa.html#parsing_mpadded-length
# wrapped for display
 = xsd:string {
  pattern = '\s*([\+\-]?[0-9]*([0-9]\.?|\.[0-9])[0-9]*\s*((%?\s*(height|depth|width)?)|
         e[mx]|in|cm|mm|p[xtc]|((negative)?((very){0,2}thi(n|ck)|medium)mathspace))?)\s*' }

The second parenthesis group looks compulsory.
(Reporter)

Comment 2

2 years ago
Thank you for looking into this. My bad, I read the spec too quickly and was confused by the current incorrect/partial implementation in WebKit.

I updated http://tests.mathml-association.org/mathml/relations/css-styling/lengths-2.html
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.