Closed Bug 427198 Opened 17 years ago Closed 5 years ago

Register nsMsgServiceProviderService with the Cycle collector, or remove rdf ispdata and replace by another version.

Categories

(MailNews Core :: Backend, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: standard8, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: memory-leak)

From bug 426963, we are leaking a nsMsgServiceProviderService and (probably) some RDF items related to it. Though I'm guessing it only happens when you bring up the account manager dialog. We should either fix the service not to leak by registering it with the cycle collector (similar to bug 387709, easy fix), or replace the rdf ispdata options with a js or widget based solution (seeing as we're generally trying to get rid of rdf).
Flags: blocking-thunderbird3?
Bug 418693 is for replacing rdf ispdata.
(In reply to comment #1) > Bug 418693 is for replacing rdf ispdata. > Added as dependency, I think we should either fix that or this, before TB 3.
Depends on: 418693
a one-time small leak doesn't warrant blocking status IMO, but that doesn't mean we shouldn't fix it.
Flags: wanted-thunderbird3+
Flags: blocking-thunderbird3?
Flags: blocking-thunderbird3-
Product: Core → MailNews Core
Assignee: nobody → bugzilla
Priority: -- → P3
Target Milestone: --- → Thunderbird 3.0b1
Target Milestone: Thunderbird 3.0b1 → Thunderbird 3.0b2
Target Milestone: Thunderbird 3.0b1 → Thunderbird 3.0rc1
Was this fixed by bug 422814 too?
(In reply to comment #4) > Was this fixed by bug 422814 too? Nope, nsMsgServiceProviderService is still present and its reference counting setup hasn't been adjusted to use the cycle collector.
As bug 418693 is WONTFIX apparently, can the "easy fix" as described in comment #0 be done?
I remember looking at things similar to this before, iirc the hardest part is working out what macros to use where. I'm not actively working on this, nor do I really think it is worth me picking up in the next period of time. Therefore resetting assignee.
Assignee: bugzilla → nobody
Priority: P3 → --
Target Milestone: Thunderbird 3.0rc1 → ---
If this and bug 387709 haven't been done then it can't be terribly important => minor. Unless someone thinks otherwise. :)
Severity: normal → minor

nsMsgServiceProviderService doesn't exit anymore. Nor does rdf ispdata. ->WFM

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.