cloning virtual folder views leaks

RESOLVED FIXED in Thunderbird 3.0rc1

Status

defect
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

Trunk
Thunderbird 3.0rc1
x86
Windows NT
Bug Flags:
blocking-thunderbird3 +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [no l10n impact])

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Posted patch proposed fixSplinter Review
Cloning virtual folder views, which we do when the user switches between thread and flat, for example), leaks the view and holds the db open. Fix upcoming.

Because the view is added as a listener to the db, we need to call Close() so the view can unwind the ownership.

I'll be attaching a patch to bug 513315 that makes this leak easy to see.
Flags: blocking-thunderbird3+
Attachment #403797 - Flags: superreview?(bugzilla)
Attachment #403797 - Flags: review?(bugzilla)
Whiteboard: [no l10n impact]
(Assignee)

Comment 1

10 years ago
if you run the xpcshell test mail/base/test/ test_viewWrapper_virtualFolder.js with the patch for bug 513315 applied, you'll see the leak, and with this patch, you'll see the leak fixed.
Status: NEW → ASSIGNED
Whiteboard: [no l10n impact] → [no l10n impact][has patch for r/sr standard8]
Attachment #403797 - Flags: superreview?(bugzilla)
Attachment #403797 - Flags: superreview+
Attachment #403797 - Flags: review?(bugzilla)
Attachment #403797 - Flags: review+
(Assignee)

Comment 2

10 years ago
fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Whiteboard: [no l10n impact][has patch for r/sr standard8] → [no l10n impact]
You need to log in before you can comment on or make changes to this bug.