Open Bug 2077861 Opened 18 days ago Updated 1 day ago

"message" event listener in mailWindow.js doesn't handle messages which contain a status message and a meteor request at the same time

Categories

(Thunderbird :: General, defect)

Thunderbird 150
defect

Tracking

(thunderbird_esr153 wontfix, thunderbird159 wontfix)

REOPENED
160 Branch
Tracking Status
thunderbird_esr153 --- wontfix
thunderbird159 --- wontfix

People

(Reporter: yury_ivanovich, Assigned: mkmelin)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 external file)

Adding console.log(event.data) here:
https://searchfox.org/comm-central/rev/91e567edd651d4405c74a24f97ac7defc6c95a86/mail/base/content/mailWindow.js#357
It shows:
Object { statusMessage: "test: Compacting folder Inbox…", meteors: "start-meteors" }
Object { statusMessage: "Done compacting (approx. 415 KB saved).", meteors: "stop-meteors" }

Since the meteor handling is in the else-block, it's not executed for those messages.

The "message" listener in mailWindow.js only processed progress and meteors
when there was no status message, so a status update combined with
start-meteors or stop-meteors (folder reparse, compaction, Exchange) never
reached startMeteors()/stopMeteors().

Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Severity: -- → S4
Target Milestone: --- → 160 Branch

Pushed by vineet@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/050dda1fab6d
Handle meteors requests that come with a status message. r=arschmitz

Status: ASSIGNED → RESOLVED
Closed: 1 day ago
Resolution: --- → FIXED
Backout by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/387a1b0b7899 Backed out changeset 67bef47406c0 for test failures
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: