Closed Bug 1247401 Opened 4 years ago Closed 4 years ago

remove nsISupports implementation from SVGTests

Categories

(Core :: SVG, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

SVGTests is purely a mixin class for classes that get their nsISupports
bits from someplace else.  Having it implement nsISupports means that
the classes which inherit from it wind up with multiple refcount
members, which is a waste of space.  By removing nsISupports
implementation from SVGTests, we save a bit of space and potential
confusion along the way.
Comment on attachment 8718065 [details] [diff] [review]
remove nsISupports implementation from SVGTests

Thanks! r=me
Attachment #8718065 - Flags: review?(dholbert) → review+
https://hg.mozilla.org/mozilla-central/rev/6cdeef48b9d6
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.