20757 - parameterless window.open() doesn't, and prevents app quit

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[David.Chappell]

It is my understanding that if the first argument to windows.open() is an empty
string or null, then a new window should be created but no document should be
loaded.  M11 attempts to load a document with an empty name.  (Which is likely
to be an index.html file, a directory listing, or an error page.)

Comment 1

[Mike McCabe]

Reassiging to the DOM component.  window.open isn't in the core JavaScript
engine.

Comment 2

[vidur (gone)]

In an attempt to get my bug list in order again, marking all the bugs I have
currently as ASSIGNED.

Comment 3

[Matt Bissiri]

Attachment: patch to fix bug 20757

Comment 4

[vidur (gone)]

Reassigning to danm, since he's now defacto owner of window.open().

Comment 5

[Dan M]

window.open() without parameters should open a new, blank window. Mozilla doesn't.
Added bonus: mozilla will refuse to shut down after attempting to open such a window.
Neato.

Comment 6

[Dan M]

I've applied the patch submitted above by bissiri; it explicitly skips the URL load if no URL
parameter was specified. Fixes the original complaint and also solves the secondary symptom
I mentioned just above (app refuses to quit). Thanks, masked man!

Comment 7

[Dan M]

Oops. Patch was actually very evil. Backing out for now.

Comment 8

[Dan M]

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

Comment 9

[Dan M]

After resubmitting a second patch from Matthew, we seem to get all the good behaviour
without the evil.  Not really sure what went wrong last time. Perhaps I messed up
a brace or something in my last attempt. Seems good this time. Sorry for the confusion.