346114 - New window from bookmarklet disappears to the background

Status: RESOLVED DUPLICATE of bug 232605

(Core :: Widget: Win32, defect)

Description

[Ria Klaassen (not reading all bugmail)]

Steps to reproduce:

- In a new profile with default settings (including browser.link.open_newwindow.restriction to 2) paste this in the location bar and press enter:  javascript:void(window.open("", "", "dependent=yes"))
- In a flash you'll see the window appear and quickly disappear to the background.

This is a regression from Bug 344257.
I see this bug on both trunk and branch.

When I change the bookmarklet to dependent=no I still see the same behaviour and when I remove the dependent=yes it opens the window in a new tab.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Ria, what about:

  javascript:void(window.open("", "", "height=100,width=100"))

?  Does that also open in the bakground?  And if so, does that behavior also change with bug 344257?

What about:

    javascript:void(window.open("", "", toolbar=yes"))

?

Comment 2

[Ria Klaassen (not reading all bugmail)]

(In reply to comment #1)
> Ria, what about:
> 
>   javascript:void(window.open("", "", "height=100,width=100"))
>

This opens a very small window in the background in the same way.
 
> ?  Does that also open in the bakground?  And if so, does that behavior also
> change with bug 344257?

No this seems the same. Pop-up was a bit higher before the patch.

> 
> What about:
> 
>     javascript:void(window.open("", "", toolbar=yes"))
> 
> ?
> 
This does nothing.

Comment 3

[Ria Klaassen (not reading all bugmail)]

(In reply to comment #0)
>
I see an error appear when I hit enter:
Error: this.editor has no properties
Source File: chrome://global/content/bindings/textbox.xml
Line: 131

Comment 4

[Boris Zbarsky [:bzbarsky]]

Er, I meant:

    javascript:void(window.open("", "", "toolbar=yes"))

It sounds like there is a general problem with opening popups in the background, basically.  That needs fixing.  Do we know when that regressed (using the width=100,height=100 testcase)?

All bug 344257 did was make "dependent=yes" effectively a no-op for non-chrome windows, really.

Comment 5

[Ria Klaassen (not reading all bugmail)]

(In reply to comment #4)
> 
Sorry. No, the same behaviour: everything opens in the background with the same error.

Comment 6

[Ria Klaassen (not reading all bugmail)]

The error is unrelated in any case; coincidence.

Comment 7

[Ria Klaassen (not reading all bugmail)]

And the pop-up has always opened in the background using
javascript:void(window.open("", "", "height=100,width=100"))
This is not a regression.

Comment 8

[Boris Zbarsky [:bzbarsky]]

Ah, I see.  In that case this is nice to fix, but probably not urgent...

Comment 9

[Jesse Ruderman]

Isn't this a dup of bug 232605, a regression from 2001?

Comment 10

[Boris Zbarsky [:bzbarsky]]

Ah, yes.

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