Closed
Bug 545886
Opened 14 years ago
Closed 14 years ago
Make summarization code update the message display's messageLoading/messageLoaded
Categories
(Thunderbird :: Testing Infrastructure, defect)
Thunderbird
Testing Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.1b1
People
(Reporter: rain1, Assigned: rain1)
References
Details
Attachments
(1 file)
7.68 KB,
patch
|
asuth
:
review+
|
Details | Diff | Splinter Review |
I don't think having multiple summarizations active for the same message display is something that can happen, so this should be safe. This should fix timeouts with both bug 542249 and bug 545626.
Attachment #426700 -
Flags: review?(bugmail)
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → sid.bugzilla
Status: NEW → ASSIGNED
Comment 1•14 years ago
|
||
Comment on attachment 426700 [details] [diff] [review] patch, v1 Well structured and nice and clean! Thanks!
Attachment #426700 -
Flags: review?(bugmail) → review+
Assignee | ||
Comment 2•14 years ago
|
||
http://hg.mozilla.org/comm-central/rev/2d353660d7d1 I'm going to mark this as in-testsuite+ as soon as bug 540110 lands, as with that patch tests won't succeed unless this works properly.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Target Milestone: --- → Thunderbird 3.1b1
You need to log in
before you can comment on or make changes to this bug.
Description
•