738163 - resource://gre/modules/mailServices.js should be resource:///modules/mailServices.js.

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Philip Chee]

Mail Services was introduced in Bug 700899 - Port |Bug 193281 - Ability to import/save e-mail messages from .eml text files to folders (Message|Copy)|.
http://hg.mozilla.org/comm-central/annotate/dca487087cf1/suite/mailnews/mailWindowOverlay.js#l52
http://hg.mozilla.org/comm-central/rev/3ab8b701b57e#l3.12

See: Bug 736495 (Some scripts are importing modules with resource:///gre instead of resource:/// and vice-versa) for why resource://gre/modules/mailServices.js should be resource:///modules/mailServices.js.

Comment 1

[Philip Chee]

Attachment: Patch v1.0 one liner.

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 608268 [details] [diff] [review]
Patch v1.0 one liner.

Review of attachment 608268 [details] [diff] [review]:
-----------------------------------------------------------------

Would you care to fix
/mailnews/imap/test/unit/test_saveTemplate.js
    * line 44 -- Components.utils.import("resource://gre/modules/mailServices.js");
too?

::: suite/mailnews/mailWindowOverlay.js
@@ +53,1 @@
>  Components.utils.import("resource:///modules/folderUtils.jsm");

Nit: sort it after folderUtils.jsm.

Comment 3

[Philip Chee]

> Would you care to fix...
Please file a separate bug for /mailnews as they have different reviewers and review rules.

Comment 4

[Philip Chee]

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/03468b543189

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to Serge Gautherie (:sgautherie) from comment #2)
> /mailnews/imap/test/unit/test_saveTemplate.js
>     * line 44 --
> Components.utils.import("resource://gre/modules/mailServices.js");

Bug 739782 Submitted