Closed
Bug 477588
Opened 15 years ago
Closed 15 years ago
If Send Unsent Messages fails, then you can't re-attempt in the same session.
Categories
(MailNews Core :: Networking: SMTP, defect)
MailNews Core
Networking: SMTP
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b2
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
7.02 KB,
patch
|
Bienvenu
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Steps to repeat: 1) Set up an smtp failure such that the server is missing. 2) Write a message, select to send later. 3) Select Send Unsent Messages 4) Acknowledge failure 5) Try to select Send Unsent Messages => You can't. The problem is that the failure route doesn't call StartNextMailFileSend, which is related to bug 151364. However, whenever we stop sending unsent mail, we should make sure we clear the mSendingProgress flag. I noticed a couple of instances where if StartNextMailFileSend fails, NotifyListenersOnStopSending is called, so it seems the correct thing to do is to reset mSendingProgress in the NotifyListenersOnStopSending function. Patch attached with test case.
Attachment #361262 -
Flags: superreview?(bienvenu)
Attachment #361262 -
Flags: review?(bienvenu)
Updated•15 years ago
|
Attachment #361262 -
Flags: superreview?(bienvenu)
Attachment #361262 -
Flags: superreview+
Attachment #361262 -
Flags: review?(bienvenu)
Attachment #361262 -
Flags: review+
Comment 1•15 years ago
|
||
Comment on attachment 361262 [details] [diff] [review] The fix + // Ensure we have a local mail account, an normal account and appropriate + // servers and identities. should be "a normal account"
Assignee | ||
Comment 2•15 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/e79acef645e3
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•