Implementation & test case for the mathcolor and mathbackground attributes defined in the MathML 3 w3c Draft: http://www.w3.org/TR/MathML3/chapter3.html#presm.presatt
Created attachment 60957 [details] Patch
Comment on attachment 60957 [details] Patch WebCore/mathml/MathMLElement.cpp:4 + * Copyright (C) 2010 François Sausset (sausset@gmail.com). All rights reserved. There seems to be some unicode issues here. 62 if (attr->name() == MathMLNames::mathcolorAttr) { 63 addCSSProperty(attr, CSSPropertyColor, attr->value()); 64 } else if (attr->name() == MathMLNames::mathbackgroundAttr) { 65 addCSSProperty(attr, CSSPropertyBackgroundColor, attr->value()); 66 67 } else 68 StyledElement::parseMappedAttribute(attr); These one liners should not have braces. There is also a strange newline before the last else. Apart from this the patch looks fine. Please fix these things before committing.
Created attachment 60962 [details] Revised Patch Correct the style issues.
Comment on attachment 60957 [details] Patch Cleared Kenneth Rohde Christiansen's review+ from obsolete attachment 60957 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Comment on attachment 60962 [details] Revised Patch Clearing flags on attachment: 60962 Committed r62968: <http://trac.webkit.org/changeset/62968>
All reviewed patches have been landed. Closing bug.