98785 - window.open(..., '_blank') finds extant window in embedded apps

Status: RESOLVED FIXED

(Core :: DOM: Navigation, defect)

Description

[Dan M]

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.

Comment 1

[Dan M]

Attachment: teach WindowWatcher about _blank windows

Comment 2

[Dan M]

(to test this, once I get bug 97514 closed again, just try doing anything that 
opens a _blank window. like bug 98732, for instance.)

Comment 3

[rpotts (gone)]

hey dan,

we also seem to support (at least sometimes) a window target called '_new'.  Do
we need to check for '_new' too?

Comment 4

[Dan M]

_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.