Closed Bug 527201 Opened 11 years ago Closed 7 years ago
ML attributes should not be accessible from DOM
MathML use several private attributes to implement "columnalign", "columnline", "rowalign", "rowline" (mtable attributes) and "fontstyle" (in particular to make <mi>x</mi> italic). They are used to apply CSS style via mathml.css. These attributes are currently accessible from DOM while they are not standard. I try to fix bug 475518 to see whether following the right syntax _moz* helps, but apparently it does not work. See also http://groups.google.fr/group/mozilla.dev.tech.mathml/browse_thread/thread/fd3e7defcdfb2b4f
(In reply to comment #0) > Created attachment 410943 [details] > Testcase It seems that this bug is fixed on trunk. At least, the private attributes are no longer caught by the testcase...
There are still 12 test cases in the html5lib test suite that show this problem. E.g. <!DOCTYPE html><body><table><math><mi>foo</mi></math></table> Attaching a patch that reveals these when running parser/htmlparser/tests/mochitest/test_html5_tree_construction_part2.html
Update the reduced testcase.
Attachment #410943 - Attachment is obsolete: true
(In reply to Frédéric Wang (:fredw) from comment #4) > Created attachment 490486 [details] > Testcase > > Update the reduced testcase. Note: the test is not valid MathML because rowalign and columnalign attributes are reversed. But it detects the private _moz attributes anyway.
Comment on attachment 8347313 [details] [diff] [review] Patch V2 After bugs 731667 and 114365, all the private _moz attributes have been removed. So I believe this patch can now be taken.
Attachment #8347313 - Flags: review?(hsivonen)
Comment on attachment 8347313 [details] [diff] [review] Patch V2 Awesome! Thank you.
Attachment #8347313 - Flags: review?(hsivonen) → review+
Assignee: nobody → fred.wang
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.