Closed Bug 317027 Opened 20 years ago Closed 20 years ago

Classinfo is incorrect for some SVG nodes

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: longsonr)

References

Details

Attachments

(1 file, 1 obsolete file)

For example: 2691 DOM_CLASSINFO_MAP_BEGIN(SVGTSpanElement, nsIDOMSVGTSpanElement) 2692 DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGTextPositioningElement) 2693 DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGTextContentElement) 2694 DOM_CLASSINFO_MAP_END should have either DOM_CLASSINFO_SVG_ELEMENT_MAP_ENTRIES or DOM_CLASSINFO_SVG_GRAPHIC_ELEMENT_MAP_ENTRIES added; as things stand it doesn't expose nsIDOM3Node, for example, so its baseURI property is undefined... Should probably check other SVG nodes for similar issues.
Blocks: 316959
I have been through http://www.w3.org/TR/SVG/ and tried to compare it to what we have in this file and this is what I came up with. I think this also fixes bug #313078.
Attachment #203823 - Flags: review?(tor)
Comment on attachment 203823 [details] [diff] [review] try to get mozilla to match the SVG 1.1 specification > DOM_CLASSINFO_MAP_BEGIN(SVGImageElement, nsIDOMSVGImageElement) > DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGImageElement) > DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGURIReference) >- DOM_CLASSINFO_SVG_GRAPHIC_ELEMENT_MAP_ENTRIES >+ DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGStylable) >+ DOM_CLASSINFO_SVG_ELEMENT_MAP_ENTRIES > DOM_CLASSINFO_MAP_END I don't think this change is correct - svg.idl says that <image> is supposed to be transformable. > DOM_CLASSINFO_MAP_BEGIN(SVGMarkerElement, nsIDOMSVGMarkerElement) > DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGMarkerElement) >+ DOM_CLASSINFO_MAP_ENTRY(nsIDOMSVGStylable) > DOM_CLASSINFO_SVG_ELEMENT_MAP_ENTRIES > DOM_CLASSINFO_MAP_END Should also have nsIDOMSVGFitToViewBox.
Attachment #203823 - Flags: review?(tor) → review-
Attachment #203823 - Attachment is obsolete: true
Attachment #203913 - Flags: superreview?(bzbarsky)
Attachment #203913 - Flags: review?(tor)
Attachment #203913 - Flags: review?(tor) → review+
Attachment #203913 - Flags: superreview?(bzbarsky) → superreview+
Robert, do you need that checked in?
Boris, I would like this checked in please. You chould also mark bug #313078 fixed once you do this I think.
Assignee: general → longsonr
Blocks: 313078
Checked in.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: