Closed
Bug 249607
Opened 21 years ago
Closed 21 years ago
Need to handle case of multiple deferred accounts downloading new mail at startup for global inbox
Categories
(MailNews Core :: Networking: POP, defect)
MailNews Core
Networking: POP
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
(Keywords: fixed-aviary1.0)
Attachments
(1 file)
|
3.08 KB,
patch
|
mscott
:
superreview+
|
Details | Diff | Splinter Review |
If multiple deferred accounts get new mail at startup, all but the first will
probably not be able to get the folder lock. If biff downloads new mail, and
multiple deferred accounts have the same interval, they could also contend over
the folder.
| Assignee | ||
Comment 1•21 years ago
|
||
We should be able use nsIPop3IncomingServer::DownloadMailFromServers which
chains the download of new mail for multiple pop3 servers.
| Assignee | ||
Comment 2•21 years ago
|
||
so in theory the code that checks for accounts to get new mail at startup for
could check for pop3 accounts and/or deferred accounts, and use
nsIPop3IncomingServers::DownloadMailFromServers. I don't like special casing
pop3 servers here, but I think we might already be special casing based on
server type.
If a deferred server also defers get new mail, then we don't have to put it in
the list, if the deferred to account is also set to check for new mail at
startup (which won't be the default case, since the local folders account can't
be set to check for new mail at startup).
| Assignee | ||
Comment 3•21 years ago
|
||
coalesce startup downloads by destination folder
| Assignee | ||
Comment 4•21 years ago
|
||
Comment on attachment 153955 [details] [diff] [review]
proposed fix
I'll remove that first dump...
Attachment #153955 -
Flags: superreview?(mscott)
Updated•21 years ago
|
Attachment #153955 -
Flags: superreview?(mscott) → superreview+
| Assignee | ||
Comment 5•21 years ago
|
||
fixed
Updated•21 years ago
|
Product: MailNews → Core
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•