Closed Bug 397804 Opened 13 years ago Closed 13 years ago

pages loading feeds as content can trigger a leak

Categories

(Firefox Graveyard :: RSS Discovery and Preview, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sayrer, Assigned: sayrer)

References

()

Details

(Keywords: memory-leak)

Attachments

(2 files, 1 obsolete file)

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: 13 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.