Closed
Bug 98785
Opened 24 years ago
Closed 24 years ago
window.open(..., '_blank') finds extant window in embedded apps
Categories
(Core :: DOM: Navigation, defect)
Tracking
()
RESOLVED
FIXED
mozilla0.9.5
People
(Reporter: danm.moz, Assigned: danm.moz)
Details
Attachments
(1 file)
|
791 bytes,
patch
|
Details | Diff | Splinter Review |
Oops. Why the heck had no one noticed this? If your app was using WindowWatcher
as it was supposed to, _blank wasn't properly detected, and would find an extant
window. Mozilla has been fine because until recently, WindowWatcher wasn't
really hooked up in Mozilla.
(to test this, once I get bug 97514 closed again, just try doing anything that
opens a _blank window. like bug 98732, for instance.)
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.5
|
||
Comment 3•24 years ago
|
||
hey dan,
we also seem to support (at least sometimes) a window target called '_new'. Do
we need to check for '_new' too?
_new is supported inconsistently, and not in the code from which I cribbed this
patch, but !what the heck! supporting _new as well is probably step in the
direction of consistency. And standards hell. But we were already there. Patch is
in, with _new.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•