114473 - File|Save as File broken in newmssg/reply/forward in compose window

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect, P1)

Description

[Jim Avera]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.6+) Gecko/20011207
BuildID:    2001120708

While composing a mail message,
File->Save as File... does not appear to do anything.
A file-selection dialog does not appear.

Reproducible: Always
Steps to Reproduce:
1.Start   mozilla -mail
2.Open an existing inbound message
3.Click Reply
4.Do File->Save as File...

Actual Results:  Nothing happens

Expected Results:  File-selection dialog should appear

Comment 1

[Boris Zbarsky [:bzbarsky]]

not composition

Comment 2

[Sheela Ravindran]

Changing os to all. Nominating 
Save as file does not bring up the file selection dlg only when you are trying
to save replied or forwarded message

Save as file brings up the file selection dlg from the 3 pane, stand alone
message window.

Comment 3

[Sheela Ravindran]

changing the summary

Comment 4

[Sheela Ravindran]

Bug 114874 is a recent regression which has problem saving page. 

Comment 5

[Jean-Francois Ducarroz]

Here is the error:

SaveAsFile from XUL
************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "'[JavaScript Error: "editorShell.saveDocument is not a function"
{file: "chrome://messenger/content/messengercompose/
MsgComposeCommands.js" line: 1677}]' when calling method:
[nsIController::doCommand]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIP
T_ERROR_WITH_DETAILS)"  location: "JS frame ::
chrome://global/content/globalOverlay.js :: goDoCommand :: line 67"  data: yes]
************************************************************
An error occurred executing the cmd_saveAsFile command

Looks like the editor api as changed! reassign to varada

Comment 6

[Jim Avera]

Save As->File also does not work when composing a brand-new mail
message started with File->New Message (in Linux build 2001123108).

That is, the bug exists not just when starting with Reply
or a forwarded message.

(File->Save As->File... does not pop up a dialog; nothing on stdout/stderr)

Comment 7

[Sheela Ravindran]

Reporter,
If you see the summary, I did include newmssg in which means it save as file is
not working when new message compose.  

Comment 8

[Sheela Ravindran]

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

Comment 9

[scottputterman]

reassigning to ducarroz.

Comment 10

[Jean-Francois Ducarroz]

probably an editor api change that hasn't reported in msg compose...

Comment 11

[Jean-Francois Ducarroz]

Attachment: Proposed fix, v1

Comment 12

[Cavin Song]

Comment on attachment 65059 [details] [diff] [review]
Proposed fix, v1

r=cavin.

Comment 13

[David :Bienvenu]

Comment on attachment 65059 [details] [diff] [review]
Proposed fix, v1

sr=bienvenu

Comment 14

[Jaime Rodriguez, Jr.]

Adding nsbeta1 keyword to all regressions so they *get some love* and attention.

Comment 15

[scottputterman]

Fixing a broken feature seems worth getting into 0.9.8 so moving back.

Comment 16

[Christopher Blizzard (:blizzard)]

a=blizzard on behalf of drivers for 0.9.8

Comment 17

[Jean-Francois Ducarroz]

Checked in and fixed

Comment 18

[Sheela Ravindran]

Build id: 2002-01-26-06 win98, linux
2002-01-24-06 mac os 10.1

verified save as file from 3 pane, stand alone window, new message from compose