pages loading feeds as content can trigger a leak

RESOLVED FIXED

Status

defect
RESOLVED FIXED
12 years ago
7 months ago

People

(Reporter: sayrer, Assigned: sayrer)

Tracking

({memory-leak})

Dependency tree / graph
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(2 attachments, 1 obsolete attachment)

This looks like it's actually the FeedConverter that leaks when something is really bogus. There's a bug in the toolkit parser that causes it to send things twice, but we shouldn't leak no matter what.
doesn't fix the leak yet, but probably necessary to do so
Is this related to bug 397206?
So far it looks like something different.
This makes sense. FeedConverter.js has a reference to the nsParser (as an nsIStreamListener).
Attachment #282587 - Attachment is obsolete: true
Attachment #282616 - Flags: review?(jst)
Attachment #282616 - Flags: review?(jst) → review+
Attachment #282616 - Flags: superreview?(peterv)
Attachment #282616 - Flags: superreview?(peterv) → superreview+
Though there are macros that implement all the CC stuff in a single macro when all the members are nsCOMPtrs, I can never remember the name though.
Attachment #282616 - Flags: approval1.9?
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Keywords: mlk
Attachment #282765 - Flags: superreview?(jonas)
Attachment #282765 - Flags: review?(jonas)
Attachment #282765 - Flags: superreview?(jonas)
Attachment #282765 - Flags: superreview+
Attachment #282765 - Flags: review?(jonas)
Attachment #282765 - Flags: review+
Attachment #282765 - Flags: approval1.9+
Flags: in-testsuite?
Comment on attachment 282765 [details] [diff] [review]
use the short macro in nsParser

Landed this fixup.
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.