Open
Bug 117130
Opened 23 years ago
Updated 16 years ago
Mail Startup: delay sidebar loading until after the password dialog is dismissed.
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
NEW
People
(Reporter: cavin, Unassigned)
References
(Depends on 1 open bug)
Details
(Keywords: perf)
It seems that we should delay sidebar loading until after the password dialog is
dismissed (either users login correctly or cancel the dialog) because the
password dialog is modal and users can't click on the sidebar anyway.
The test result from the 12-25 commercial build indicates that this change will
probably save us 0.2 seconds on startup (ie, the difference between starting up
mailnews with and without sidebar is about 0.2 second). It's an unofficial
number but it gives a good indication of how much improvement we'll get on this
change.
Reporter | ||
Comment 1•23 years ago
|
||
Moving it to 0.9.8.
Keywords: perf
Target Milestone: --- → mozilla0.9.8
Comment 2•23 years ago
|
||
this should not be the case if we add a newsgroup account.. cause it doesn't
display sidebar now, after newsgroup account creation. see attachment 64655 [details].
Comment 3•23 years ago
|
||
But I do get Sidebar to show content on hitting F9 twice as in bug 116094
Comment 6•23 years ago
|
||
I can reproduce this bug,but i think it's shouldn't mailnews bug,because i can
reproduce this bug ,when i open a web page (authorization required),sidebar can
not load.you can see follow code in extensions/wallet/src/nsWalletService.cpp
NS_IMETHODIMP
nsSingleSignOnPrompt::PromptPassword(const PRUnichar *dialogTitle, const
PRUnichar *text,
const PRUnichar *passwordRealm, PRUint32
savePassword,
PRUnichar **pwd, PRBool *_retval)
{
nsresult rv;
nsCAutoString realm;
realm.AssignWithConversion(passwordRealm); // XXX should be PRUnichar*
rv = SINGSIGN_PromptPassword(dialogTitle, text, pwd,
realm.get(), mPrompt, _retval, savePassword);
return rv;
}
Updated•20 years ago
|
Product: Browser → Seamonkey
Updated•19 years ago
|
Assignee: sspitzer → mail
Comment 8•18 years ago
|
||
irrelevant today -> WONTFIX per bug 26455 comment 40.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WONTFIX
Comment 9•18 years ago
|
||
> irrelevant today -> WONTFIX per bug 26455 comment 40.
1. MailNews Sidebar will come back for SeaMonkey.
2. Comment #6 sees this for non-mail as well (but that probably should be another bug).
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Updated•16 years ago
|
Status: REOPENED → NEW
QA Contact: esther
You need to log in
before you can comment on or make changes to this bug.
Description
•