Closed Bug 722920 Opened 9 years ago Closed 9 years ago

Need a utility function to copy message(s) in file into local inbox

Categories

(MailNews Core :: Testing Infrastructure, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 13.0

People

(Reporter: hiro, Assigned: hiro)

Details

Attachments

(2 files)

Some unit tests in mailnews/local/test/unit have its own copy lister to copy message(s) in file. Those listeners should be unified in head_maillocal.js

I will post a patch soon.
Attachment #593258 - Flags: review?(mconley)
Comment on attachment 593258 [details] [diff] [review]
Proposed fix
[Checked in: Comment 4]

I'm gonna steal this review since I'm looking at a related patch.
Attachment #593258 - Flags: review?(mconley) → review?(dbienvenu)
Comment on attachment 593258 [details] [diff] [review]
Proposed fix
[Checked in: Comment 4]

thx for the patch
Attachment #593258 - Flags: review?(dbienvenu) → review+
Status: NEW → ASSIGNED
Keywords: checkin-needed
Assignee: nobody → hiikezoe
Checked in: http://hg.mozilla.org/comm-central/rev/22c18ee522cb
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
Attachment #600776 - Flags: review?(dbienvenu) → review+
Attachment #593258 - Attachment description: Proposed fix → Proposed fix [Checked in: Comment 4]
Comment on attachment 600776 [details] [diff] [review]
(Bv1) test_over4GBMailboxes.js: Remove leftover 'copyService'
[Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/1940b4386045
Attachment #600776 - Attachment description: (Bv1) test_over4GBMailboxes.js: Remove leftover 'copyService' → (Bv1) test_over4GBMailboxes.js: Remove leftover 'copyService' [Checked in: Comment 6]
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.