Closed
Bug 481065
Opened 16 years ago
Closed 16 years ago
local folder db's opened on startup
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b3
People
(Reporter: Bienvenu, Assigned: Bienvenu)
References
(Blocks 1 open bug)
Details
(Keywords: perf, regression)
Attachments
(1 file)
(deleted),
patch
|
standard8
:
review+
standard8
:
superreview+
|
Details | Diff | Splinter Review |
fixing bug 469448 (or, rather, fixing it w/o breaking tests) led to a regression where we opened up folder dbs for sub-folders on startup. This fixes that regression.
Assignee | ||
Comment 1•16 years ago
|
||
the fix is to move the code that forces us to open the db into the same if clause that we hit if the folder doesn't exist (which is what the test code runs into).
Attachment #365060 -
Flags: superreview?(bugzilla)
Attachment #365060 -
Flags: review?(bugzilla)
Updated•16 years ago
|
Attachment #365060 -
Flags: superreview?(bugzilla)
Attachment #365060 -
Flags: superreview+
Attachment #365060 -
Flags: review?(bugzilla)
Attachment #365060 -
Flags: review+
Assignee | ||
Comment 2•16 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 4•16 years ago
|
||
likely Bug 491130 is fixed by this.
(can't remember if bienvenu and I discussed tests on IRC, so....)
Is the best approach to detect regression like to rely on TS tests? If so, we need to make sure TS tests have sufficient subfolders. And if not, do what type of tests do we need?
Flags: in-testsuite?
Target Milestone: --- → Thunderbird 3.0b3
Assignee | ||
Comment 5•16 years ago
|
||
Do the TS tests actually create msf files for folders, or run against profiles that have .msf files? This bug in particular was about opening .msf files that existed on startup, which is something that afaik, none of our tests are good at catching.
Updated•14 years ago
|
Blocks: tb-startupperf
You need to log in
before you can comment on or make changes to this bug.
Description
•