12345 - [DOGFOOD] Unable to Forward a message received as an Inline page or an attachment

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P1)

Description

[marina]

Steps to reproduce:
-sent to your testing account a Web page (http://home.netscape.com) from 4.x;
-get it in 5.0;
-click Forward, click Send;
//note: you don't get any e-mail
-now look at your MS-DOS window:in a successful sending you would read
" Looking for identity id1
GenereateAttachmentsString<>
attechment=   "
//now note : after looking for identity there is a line
Alert : [String ID...]
-you can successfully Forward a simple message with no attachments
****observerd with 1999-08-20 windows build****

Comment 1

[Phil Peterson]

Beta stopper. TFV M12. Jeff, can you look at this in Rich's absence?

Comment 2

[sol]

Needs to be fixed for PR1, so I added a note in the Status Whiteboard

Comment 3

[lchiang]

(target milestone is M11 or M12 - add to mail beta tracking bug)

Comment 4

[jefft]

Bumping up to M11. I hope we can get 15069 fixed for M11.

Comment 5

[jefft]

I am crashing when I try to forward message as attachment with today's build
because of failing to create file url.

Comment 6

[lchiang]

Marking dogfood since there is now a crash involved.

Comment 7

[leger]

Putting on PDT+ radar.

Comment 8

[jefft]

Check in partial fix to prevent crashing. nsMsgSend.cpp modified.

Comment 9

[jefft]

All required changes have been checked in. I am marking this bug as fixed.
Thanks to everyone's help.

Comment 10

[fenella]

Win_nt 4.0 (1999-11-09-11 M11)
Following the exact step, I am unable to forward and sent the attached email.
I am also able to receive it.

Comment 11

[marina]

Fenella,
i hope you meant " i am able to forward ..."

Comment 12

[rajesh kesari]

*** Bug 288759 has been marked as a duplicate of this bug. ***

Comment 13

[jagadeesh]

Needs to be fixed for PR1, so I added a note in the Status Whiteboard

Comment 14

[Legacy TBPL/Treeherder Robot]

submit_timestamp: 2014-10-15T12:28:57
log: https://treeherder.allizom.org/ui/logviewer.html#?repo=try&job_id=1991120
repository: try
who: cdawson[at]mozilla[dot]com
machine: t-xp32-ix-050
buildname: Windows XP 32-bit try opt test mochitest-1
revision: 5b4636011be6

Return code: 1

Comment 15

[Ruel Balabis]

[Tracking Requested - why for this release]:

Comment 16

[Wayne Mery (:wsmwk)]

undo spammer

Comment 17

[Andrew McCreight [:mccr8]]

Attachment: Some patch.

Comment 18

[Andrew McCreight [:mccr8]]

Comment on attachment 8665038 [details] [diff] [review]
Some patch.

Oops, sorry.

Comment 20

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a51cc8ac6ed2
[components] For https://github.com/mozilla-mobile/android-components/issues/12345 - Set id for all site permissions
https://hg.mozilla.org/integration/autoland/rev/11b05c1f48c8
[components] For https://github.com/mozilla-mobile/android-components/issues/12345 - Emit facts for when a permission prompt is shown/allowed/denied
https://hg.mozilla.org/integration/autoland/rev/96f4f49ce531
[fenix] For https://github.com/mozilla-mobile/fenix/issues/12345: catch SecurityException when calling reportFullyDrawn.
https://hg.mozilla.org/integration/autoland/rev/539427c0ae0c
[fenix] For https://github.com/mozilla-mobile/fenix/issues/12345: replace Activity.reportFullyDrawnSafe with a-c impl.