9578 - Editor: File | Close doesn't work

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[Chris McAfee]

Linux:
File | Close doesn't seem to do anything,
closing the window via the window manager works Ok.

Comment 1

[Charles Manske]

The editor is trying to use "window.close()", but as commented in navigator.js,
that won't work for windows not created in JavaScript. The old toolkit appcore
is used for the browser, but we shouldn't use that any more
I'll try to close the window in the editor appshell code.
Dan: Do you plan on making window.close() work for webshell windows not created
in JS?

Comment 2

[Dan M]

This problem has been fixed, using the ToolkitCore to close the window (after bug 7792 was fixed.)
So this bug can, strictly speaking, be closed.  I've opened another bug 9703 to track cmanske's
request above.

Comment 3

[Charles Manske]

Not so fast! It isn't really fixed until I checkin nsEditorShell.cpp with the
missing line:
    mWebShellWin->Close();
Sujay: Just leave this "fixed" and I'll add a note here when I checkin.

Comment 4

[sujay]

marking this bug verified fixed in 7/15 build.