Bug 64843
| Summary: | V8 binding: unable to use [Reflect] on some attributes | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Leo Yang <leo.yang> |
| Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | arv, cdumez, code.vineet |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Leo Yang
For V8 binding, some svg attribute like glyphRef can not be applied [Reflect] in idl file. Seems the generator converts the attribute to lower case. As a result, it will generate undefined identifier, like SVGNames::glyphrefAttr instead of SVGNames::glyphRefAttr.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Vineet Chaudhary (vineetc)
(In reply to comment #0)
> For V8 binding, some svg attribute like glyphRef can not be applied [Reflect] in idl file. Seems the generator converts the attribute to lower case. As a result, it will generate undefined identifier, like SVGNames::glyphrefAttr instead of SVGNames::glyphRefAttr.
Is this bug still valid? Do you have any example where we can use [Reflect] on attributes and generating wrong code.
For the case mentioned glyhRef part of SVGAltGlyphElement and per spec http://www.w3.org/TR/SVG/text.html#InterfaceSVGAltGlyphElement its shouldn't be [Reflect] or you have some specific reason to have it?
Chris Dumez
The latest specification does not say anything about [Reflect]:
https://svgwg.org/svg2-draft/text.html#InterfaceSVGGlyphRefElement
Closing this bug report. Feel free to reopen and comment if you feel this is wrong.
Chris Dumez
Also a V8 Bug.