Bug 254881
| Summary: | HTML parser: inline SVG tag name adjustment | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Anne van Kesteren <annevk> |
| Component: | DOM | Assignee: | Anne van Kesteren <annevk> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | cdumez, rniwa, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Anne van Kesteren
Assuming people review bug 254755 as a favorable change, we should consider doing the same for SVG tag names. Source/WebCore/dom/make_names.pl has a generic capability to create adjustXTagName where X can be any language, but we only need it for SVG, only in the HTML parser, and only for the tag names the HTML Standard lists (which should be a fixed list for eternity).
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Anne van Kesteren
Inlining the function is pretty straightforward, but removing all support for CaseAdjusted atoms from Source/WebCore/dom/make_names.pl as well is a bit more involved.
Is inlining adjustSVGTagName into HTMLTreeBuilder a good enough first step?
Radar WebKit Bug Importer
<rdar://problem/107802125>
Anne van Kesteren
Pull request: https://github.com/WebKit/WebKit/pull/38995