Closed Bug 855597 Opened 7 years ago Closed 7 years ago

Make SVGTransform not inherit nsISupports

Categories

(Core :: SVG, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla23

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Details

Attachments

(3 files)

No description provided.
Attachment #730541 - Flags: review?
Attachment #730541 - Flags: review? → review?(longsonr)
Attachment #730541 - Flags: review?(longsonr) → review+
Why do we need the renaming? Seems to me the existing names are fine. They correspond to the naming convention jwatt invented for lengthlists i.e.

DOMSVGLength is equivalent to DOMSVGTransform

and SVGTransform models this: http://www.w3.org/TR/SVG/coords.html#InterfaceSVGTransform

Any suggestions for naming Jonathan?
Jonathan and I decided we want to rename all these classes to be consistent with the rest of content and with what the dom bindings expect.
Attachment #731578 - Flags: review?(longsonr) → review?(jwatt)
Attachment #731579 - Flags: review?(longsonr) → review?(jwatt)
OK, if you've already discussed this with Jonathan, he can review it ;-)
Attachment #731578 - Flags: review?(jwatt) → review+
Attachment #731579 - Flags: review?(jwatt) → review+
Flags: in-testsuite-
OS: Mac OS X → All
Hardware: x86 → All
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.