Last Comment Bug 722920 - Need a utility function to copy message(s) in file into local inbox
: Need a utility function to copy message(s) in file into local inbox
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Testing Infrastructure (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Thunderbird 13.0
Assigned To: Hiroyuki Ikezoe (:hiro)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-31 16:09 PST by Hiroyuki Ikezoe (:hiro)
Modified: 2012-02-27 04:34 PST (History)
4 users (show)
bugzillamozillaorg_serge_20140323: in‑testsuite+
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Proposed fix [Checked in: Comment 4] (13.44 KB, patch)
2012-01-31 16:11 PST, Hiroyuki Ikezoe (:hiro)
mozilla: review+
Details | Diff | Splinter Review
(Bv1) test_over4GBMailboxes.js: Remove leftover 'copyService' [Checked in: Comment 6] (1.06 KB, patch)
2012-02-26 08:07 PST, Serge Gautherie (:sgautherie)
standard8: review+
Details | Diff | Splinter Review

Description Hiroyuki Ikezoe (:hiro) 2012-01-31 16:09:01 PST
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.
Comment 1 Hiroyuki Ikezoe (:hiro) 2012-01-31 16:11:16 PST
Created attachment 593258 [details] [diff] [review]
Proposed fix
[Checked in: Comment 4]
Comment 2 David :Bienvenu 2012-02-03 11:32:55 PST
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.
Comment 3 David :Bienvenu 2012-02-03 13:08:51 PST
Comment on attachment 593258 [details] [diff] [review]
Proposed fix
[Checked in: Comment 4]

thx for the patch
Comment 4 Mark Banner (:standard8) 2012-02-13 13:02:06 PST
Checked in: http://hg.mozilla.org/comm-central/rev/22c18ee522cb
Comment 5 Serge Gautherie (:sgautherie) 2012-02-26 08:07:52 PST
Created attachment 600776 [details] [diff] [review]
(Bv1) test_over4GBMailboxes.js: Remove leftover 'copyService'
[Checked in: Comment 6]
Comment 6 Serge Gautherie (:sgautherie) 2012-02-27 04:33:46 PST
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

Note You need to log in before you can comment on or make changes to this bug.