Closed Bug 11814 Opened 25 years ago Closed 25 years ago

From C++ using Window.Open with JSVAL_NULL takes over existing windows

Categories

(Core :: XUL, defect, P3)

Product:
Core
Component:
XUL
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED INVALID

People

(Reporter: travis, Assigned: danm.moz)

Details

When creating a window using Window.Open from C++ passing format "svs" and
window name as JSVAL_NULL causes the previous window with those parameters to be
replaced.  Calling with "sss" and window name being "" properly works creating a
new window.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → INVALID
Whatever value is passed as the window name is converted to a string, be it numeric, true string,
or JSVAL_NULL.  (And of course the JS null value is converted to the musical string "null".)
According to Brendan, this is tried-'n-true behaviour (tm) and we don't want to be changing that.
The One True Way to ensure you get a new window is to pass either nothing at all, or the string "".
What actually wants to be done here is to track down all the places in xpfe code where JSVAL_NULL
is being sent to window.open, and fix them.  Or even better, make that single C++ API function that
everyone can use, and ensure it behaves correctly.
Status: RESOLVED → VERIFIED
verified invalid
You need to log in before you can comment on or make changes to this bug.