Closed Bug 441043 Opened 17 years ago Closed 17 years ago

Change nsIMsgFolderListener folderDeleted notification to fire only for the topmost folder deleted

Categories

(MailNews Core :: Backend, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9.1a1

People

(Reporter: rain1, Assigned: rain1)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
In line with the folderMoveCopyCompleted change, this aims to change the folderDeleted notification to fire only for the topmost folder deleted. Also enables tests. Bug 439225 needs to be checked in first.
Status: NEW → ASSIGNED
Attachment #326120 - Flags: superreview?(bienvenu)
Attachment #326120 - Flags: review?(bienvenu)
Comment on attachment 326120 [details] [diff] [review] patch bug 439108 will work out, I'm sure.
Attachment #326120 - Flags: ui-review+
Attachment #326120 - Flags: review?(bienvenu)
Attachment #326120 - Flags: review+
Attachment #326120 - Flags: ui-review+
Attachment #326120 - Flags: superreview?(bienvenu)
Attachment #326120 - Flags: superreview+
Keywords: checkin-needed
Checking in mailnews/base/test/unit/test_nsIMsgFolderListenerLocal.js; /cvsroot/mozilla/mailnews/base/test/unit/test_nsIMsgFolderListenerLocal.js,v <-- test_nsIMsgFolderListenerLocal.js new revision: 1.4; previous revision: 1.3 done Checking in mailnews/base/util/nsMsgDBFolder.cpp; /cvsroot/mozilla/mailnews/base/util/nsMsgDBFolder.cpp,v <-- nsMsgDBFolder.cpp new revision: 1.351; previous revision: 1.350 done
Flags: in-testsuite+
Keywords: checkin-needed
Target Milestone: --- → mozilla1.9.1a1
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: