Closed Bug 1066483 Opened 5 years ago Closed 5 years ago

Update SVG camelCase knowledge to omit names dropped from the spec


(Core :: DOM: HTML Parser, defect)

Not set





(Reporter: hsivonen, Assigned: hsivonen)




(3 files)

Some SVG camelCase attributes have been dropped by the SVG WG and then dropped by the HTML spec. I'm not sure how good an idea it is to tweak this stuff, but since Blink has already tweaked their parser (except for filterRes), I guess it's too late to second-guess this direction.
Attached patch Java patchSplinter Review
Attached patch Gecko patchSplinter Review
The test file update comes verbatim from upstream.
Attachment #8488490 - Flags: review?(wchen)
Attachment #8488502 - Flags: review?(wchen)
Comment on attachment 8488502 [details] [diff] [review]
Gecko patch

Review of attachment 8488502 [details] [diff] [review]:

For contentStyleType and contentScriptType, we should probably remove the comments here too:

It looks like there is already a bug for removing the parsing of filterRes (bug 979472), we should also get rid of these gecko bits since they aren't being used anywhere:

r=me with those changes.
Attachment #8488502 - Flags: review?(wchen) → review+
Attachment #8488490 - Flags: review?(wchen) → review+
Duplicate of this bug: 979472
Thanks. Landed with the review comments addressed:
It looks like we have another copy of the html5lib tests in web platform tests, and it's failing due to unexpected test passes. We probably just need to delete this file:
Attachment #8505344 - Flags: review?(james)
Attachment #8505344 - Flags: review?(james) → review+
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.