1724177 - remove the cpp nsMsgSend implementation

Status: RESOLVED FIXED

(MailNews Core :: Composition, task)

Description

[Magnus Melin [:mkmelin]]

Bug 1211292 and friends are fixed. 91 has branched: we can now remove the old cpp implementation going forwards.

Comment 1

[Ping Chen [:rnons] (away)]

Attachment: Bug 1724177 - Make MessageSend.jsm the only implementation of nsIMsgSend. r=mkmelin

• Remove nsMsgSend from nsMailModule.cpp
• Remove MsgSendModuleLoader.jsm

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/d9d14fb90d34
Make MessageSend.jsm the only implementation of nsIMsgSend. r=mkmelin

Comment 3

[Ping Chen [:rnons] (away)]

Attachment: Bug 1724177 - Remove JaSend.cpp to prepare for removing nsMsgSend.cpp. r=mkmelin

Comment 4

[Ping Chen [:rnons] (away)]

Attachment: Bug 1724177 - Remove nsIMsgSend from nsMsgSend.cpp. r=mkmelin

Also remove nsIMsgAttachmentHandler and nsIURLFetcher.

Depends on D125928.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/7dd9dea88352
Remove JaSend.cpp to prepare for removing nsMsgSend.cpp. r=mkmelin
https://hg.mozilla.org/comm-central/rev/d7ac74826920
Remove nsIMsgSend from nsMsgSend.cpp. r=mkmelin

Comment 6

[Geoff Lankow (:darktrojan)]

Are we done here?

Comment 7

[Ping Chen [:rnons] (away)]

nsMsgSend.cpp still contains nsMsgAttachmentData and nsMsgAttachedFile, will split them to their own files.

Comment 8

[Ping Chen [:rnons] (away)]

Attachment: Bug 1724177 - Split out nsMsgAttachmentData and nsMsgAttachedFile, remove nsMsgSend.cpp. r=mkmelin

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/03936a89e32a
Split out nsMsgAttachmentData and nsMsgAttachedFile, remove nsMsgSend.cpp. r=mkmelin