Closed Bug 455932 Opened 12 years ago Closed 12 years ago

Add nsIMsgFolderListener notifications for news messages

Categories

(MailNews Core :: Backend, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9.1b1

People

(Reporter: rain1, Assigned: rain1)

References

Details

Attachments

(2 files)

As it says on the tin :)

- msgAdded is covered in the patch, as is msgsDeleted
- msgsMoveCopyCompleted is handled in the dest folder, and a news folder can't be a dest folder
- folderDeleted is covered by nsMsgDBFolder
Attachment #339315 - Flags: superreview?(bienvenu)
Attachment #339315 - Flags: review?(bienvenu)
Attachment #339315 - Flags: superreview?(bienvenu)
Attachment #339315 - Flags: superreview+
Attachment #339315 - Flags: review?(bienvenu)
Attachment #339315 - Flags: review+
Blocks: 430614
Keywords: checkin-needed
Checked in, changeset id: 380:322f1a754d09
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b1
I noticed that there are still more notifications than there should be. This patch batches an entire cleanup of expired/cancelled messages into one notification.
Attachment #339449 - Flags: superreview?(bienvenu)
Attachment #339449 - Flags: review?(bienvenu)
Comment on attachment 339449 [details] [diff] [review]
[checked in] Batch more notifications together

it was handy that debug code was there :-)
Attachment #339449 - Flags: superreview?(bienvenu)
Attachment #339449 - Flags: superreview+
Attachment #339449 - Flags: review?(bienvenu)
Attachment #339449 - Flags: review+
Keywords: checkin-needed
Attachment #339315 - Attachment description: patch v1 → [checked in] patch v1
Comment on attachment 339449 [details] [diff] [review]
[checked in] Batch more notifications together

Checked in, changeset id: 385:7c6c4b32c099
Attachment #339449 - Attachment description: Batch more notifications together → [checked in] Batch more notifications together
You need to log in before you can comment on or make changes to this bug.