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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.1b1

People

(Reporter: rain1, Assigned: rain1)

References

Details

Attachments

(1 file)

Attached patch patch, v1Splinter 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: nobody → sid.bugzilla
Status: NEW → ASSIGNED
Comment on attachment 426700 [details] [diff] [review]
patch, v1

Well structured and nice and clean! Thanks!
Attachment #426700 - Flags: review?(bugmail) → review+
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
Target Milestone: --- → Thunderbird 3.1b1
Er, yeah.
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: