Closed Bug 1034923 Opened 10 years ago Closed 10 years ago

AccEventGen.py code generator should not generate dangerous public destructors in refcounted classes

Categories

(Core :: Disability Access APIs, defect)

Other Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: bjacob, Assigned: mccr8)

References

Details

Attachments

(1 file)

In bug 1028588 we removed dangerous public destructors of XPCOM-refcounted classes outside of a finite whitelist, see HasDangerousPublicDestructor. Now we are going over the entries in this whitelist.

One such occurrence is in the AccEventGen.py code generator. It currently generates classes with public destructors, even though the classes are reference-counted. For that reason, in bug 1028588 we are making this code generator also generate HasDangerousPublicDestructor specializations for these classes.

But what we really want is no dangerous public destructors at all.
Component: DOM: Events → Disability Access APIs
Hah - I was looking for "accessibility".
Hmm. It seems to be okay to just make the dtor private here, which seems odd to me.
Assignee: nobody → continuation
Comment on attachment 8463439 [details] [diff] [review]
Make AccEventGen.py code generator generate classes with private destructors.

thanks!
Attachment #8463439 - Flags: review?(trev.saunders) → review+
https://hg.mozilla.org/mozilla-central/rev/2fd5026e35f1
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: