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

RESOLVED FIXED in Thunderbird 13.0

Status

MailNews Core
Testing Infrastructure
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: hiro, Assigned: hiro)

Tracking

Trunk
Thunderbird 13.0
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
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.
(Assignee)

Comment 1

6 years ago
Created attachment 593258 [details] [diff] [review]
Proposed fix
[Checked in: Comment 4]
Attachment #593258 - Flags: review?(mconley)

Comment 2

6 years ago
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 3

6 years ago
Comment on attachment 593258 [details] [diff] [review]
Proposed fix
[Checked in: Comment 4]

thx for the patch
Attachment #593258 - Flags: review?(dbienvenu) → review+
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
Keywords: checkin-needed
Assignee: nobody → hiikezoe
Checked in: http://hg.mozilla.org/comm-central/rev/22c18ee522cb
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
Created attachment 600776 [details] [diff] [review]
(Bv1) test_over4GBMailboxes.js: Remove leftover 'copyService'
[Checked in: Comment 6]
Attachment #600776 - Flags: review?(dbienvenu)
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.