Closed Bug 854629 Opened 7 years ago Closed 7 years ago

Don't declare nsISupports in SVG element classes

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Details

Attachments

(3 files)

No description provided.
Attachment #729301 - Flags: review?(Ms2ger)
Attachment #729302 - Flags: review?(Ms2ger)
Attachment #729301 - Flags: review?(Ms2ger) → review+
Want to also remove hasXPConnectImpls from the SVGElement descriptor in Bindings.conf?
Attachment #729302 - Flags: review?(Ms2ger) → review+
Comment on attachment 729304 [details] [diff] [review]
Part 2: Remove nsISupports when there is only one base class

Review of attachment 729304 [details] [diff] [review]:
-----------------------------------------------------------------

\o/
Attachment #729304 - Flags: review?(Ms2ger) → review+
(In reply to :Ms2ger from comment #3)
> Want to also remove hasXPConnectImpls from the SVGElement descriptor in
> Bindings.conf?

Good catch
For part 1, you can nix the svgelement include in that file too, right?
I meant part 0.  ;)
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.