Closed Bug 1274789 Opened 8 years ago Closed 8 years ago

namedspaces are not supported in mpadded attributes

Categories

(Core :: MathML, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID
Tracking Status
firefox49 --- affected

People

(Reporter: fredw, Unassigned)

References

()

Details

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.
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
Closed: 8 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.