Created attachment 83207 [details] Various tests of popup options Forgot to mention that this is happening on RC2 (2002051006).
Browser, not engine ---> DOM Level 0 Here is the testcase: <button onclick="window.open('', '', '')">No options</button> <button onclick="window.open('', '', 'location=yes')">location=yes</button> <button onclick="window.open('', '', 'location=no')">location=no</button> <button onclick="window.open('', '', 'menubar=yes')">menubar=yes</button> <button onclick="window.open('', '', 'menubar=no')">menubar=no</button> <button onclick="window.open('', '', 'thisstring')">thisstring</button>
Assignee: rogerl → jst
Status: UNCONFIRMED → NEW
Ever confirmed: true
QA Contact: pschwartau → desale
I thought this was the intended behaviour, but I tried on NS4x and passing an emptry string for the third argument to window.open will also open a window with no features... In any case, I have a fix for this.
Created attachment 83220 [details] [diff] [review] Proposed patch v1.0 If we get passed three arguments, also check to see if the third argument is empty. If it is, then give it a dummy argument so that we don't think that there really was no argument specified. This allows window.open() to work still with all features, but addresses the issue in this bug.
Please run this through danm before proceeding any further. I thought this was the intended behavior (memories of old bug comments...)
Yeah, I thought it was intended too... (see my comment #3). The fix is simple though if we want to take it. If not, no harm done :-)
Mass-reassigning bugs to email@example.com
Assignee: jst → dom_bugs
Assignee: general → nobody
QA Contact: desale → general
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046 Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5. If you have questions, please contact :mdaly.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.