116794 - "Open windows by themselves" disables frames

Status: VERIFIED DUPLICATE of bug 104470

(Core :: Security: CAPS, defect)

Description

[Boris 'pi' Piwinger]

Mozilla/5.0 (Windows; U; Win95; en-US; rv:0.9.7) Gecko/20011221

Choosing "open windows by themselves" also disables changing of already existing
windows. An example can be seen on:
http://www.logic.univie.ac.at/~3.14/mozilla-bugs-frames/frameset2.html

Many lusers use JavaScript to modify two or more frames in a given frameset at
the same time. This no longer works, if this option is chosen. Expected behavior
would be that only pop-ups, i.e., *new* windows are blocked.

pi

Comment 1

[Boris Zbarsky [:bzbarsky]]

To CAPS.  There is no way to easily tell the window.open() calls that target 
existing frames/windows from the ones that require new ones without actually 
trying to open the window...

Note that if the window.open() call were triggered by a user action the pref 
would _not_ block it.

Comment 2

[Mitchell Stoltz (not reading bugmail)]

I suppose we could change the disable-open-during-load function to allow the
window.open call if it's targeted at an already-open window or frame. What do
people think about that?

Comment 3

[Boris Zbarsky [:bzbarsky]]

That seems to make sense, as the point of the whole disable_open_during_load 
hack is to prevent new windows from appearing and in this case they would not 
appear.

Comment 4

[Boris 'pi' Piwinger]

This bug is also in Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.8)
Gecko/20020206. Marking OS=All.

Still annoying. How does window.open() decide if it opens a new window or reuses
an existing one? Can't this method be used here?

pi

Comment 5

[Boris Zbarsky [:bzbarsky]]

window.open() does not decide that.  It just asks the docshell to do a targeted
load and the docshell handles retargeting, creating new windows if necessary,
and the like.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Duplicate of "dom.disable_open_during_load breaks opens targeted to frames or
existing named windows"

*** This bug has been marked as a duplicate of 104470 ***

Comment 7

[Boris 'pi' Piwinger]

As usual, Boris is right.

pi