Closed
Bug 519736
Opened 16 years ago
Closed 16 years ago
cloning virtual folder views leaks
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0rc1
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
(Whiteboard: [no l10n impact])
Attachments
(1 file)
|
707 bytes,
patch
|
standard8
:
review+
standard8
:
superreview+
|
Details | Diff | Splinter 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)
|
||
Updated•16 years ago
|
Whiteboard: [no l10n impact]
|
Assignee | |
Comment 1•16 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]
|
||
Updated•16 years ago
|
Attachment #403797 -
Flags: superreview?(bugzilla)
Attachment #403797 -
Flags: superreview+
Attachment #403797 -
Flags: review?(bugzilla)
Attachment #403797 -
Flags: review+
|
|
Assignee | |
Comment 2•16 years ago
|
||
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 16 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.
Description
•