The first thing to create a mailnews URL triggers reading of the message database(s)
Categories
(MailNews Core :: Networking, defect, P2)
Tracking
(thunderbird_esr102 fixed)
Tracking | Status | |
---|---|---|
thunderbird_esr102 | --- | fixed |
People
(Reporter: darktrojan, Assigned: darktrojan)
References
(Blocks 1 open bug)
Details
(Keywords: perf, perf:startup, Whiteboard: [TM:102.3.0])
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-esr102+
|
Details | Review |
I profiled application start-up on my main profile. More than half of the first two seconds is spent reading Mork databases.
Ultimately this is triggered by the Permission Manager loading permissions into memory. Each permission URL string is turned into an nsIURI
and therefore an nsMsgMailNewsUrl
for our protocols. The creation of an nsMsgMailNewsUrl
(at least in the IMAP case) checks whether the message is available offline and therefore needs to read the .msf file.
I think delaying this one piece of information until needed will be enough to prevent the start-up performance issue.
Comment 1•2 years ago
|
||
Not only message db, but also mail accounts. This caused bug 1745033 comment 47.
Assignee | ||
Comment 2•2 years ago
|
||
I'm going to paper over this problem by initialising the permissions manager before telling the folder service that strings are ready, since if strings are not ready, this expensive code doesn't run.
I'd like to take the expensive code away from here and only run it when needed, but that seems like a complicated and potentially hazardous change, particularly given there's a one-line alternative.
Assignee | ||
Comment 3•2 years ago
|
||
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/6e2506a48931
Initialise the permission manager before telling the folder service that strings are available. r=mkmelin
Assignee | ||
Comment 5•2 years ago
|
||
Comment on attachment 9290835 [details]
Bug 1785967 - Initialise the permission manager before telling the folder service that strings are available. r=mkmelin,BenC
[Approval Request Comment]
Regression caused by (bug #): the ravages of time
User impact if declined: slower start-up than necessary
Testing completed (on c-c, etc.): in 105
Risk to taking this patch (and alternatives if risky): minimal
This should've had enough beta time by the time we release 102.3.0.
Comment 7•2 years ago
|
||
Comment on attachment 9290835 [details]
Bug 1785967 - Initialise the permission manager before telling the folder service that strings are available. r=mkmelin,BenC
[Triage Comment]
Approved for esr102
Comment 8•2 years ago
|
||
bugherder uplift |
Thunderbird 102.3.0:
https://hg.mozilla.org/releases/comm-esr102/rev/b91536145fab
Description
•