pages loading feeds as content can trigger a leak

RESOLVED FIXED

Status

defect
RESOLVED FIXED
12 years ago
5 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)

Assignee

Description

12 years ago
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.
Assignee

Comment 1

12 years ago
doesn't fix the leak yet, but probably necessary to do so

Comment 2

12 years ago
Is this related to bug 397206?
Assignee

Comment 3

12 years ago
So far it looks like something different.
Assignee

Comment 4

12 years ago
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+
Assignee

Updated

12 years ago
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.
Assignee

Updated

12 years ago
Attachment #282616 - Flags: approval1.9?
Assignee

Updated

12 years ago
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Assignee

Updated

12 years ago
Keywords: mlk
Assignee

Comment 6

12 years ago
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+

Updated

12 years ago
Flags: in-testsuite?
Assignee

Comment 7

12 years ago
Comment on attachment 282765 [details] [diff] [review]
use the short macro in nsParser

Landed this fixup.

Updated

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