15908 - Demo the pop-up

Status: VERIFIED DUPLICATE of bug 1646

(Core :: JavaScript Engine, defect, P3)

Description

[shawn]

The pop-ups don't work.  I have tried a number of them and no luck.  In addition when I tried to search for a bug report on this, M9 asked me to pick an appp to view the page when I pressed submit.  If this bug is already reported, I apologize.

Comment 1

[Mike McCabe]

This dumps core on me when I try it in debug mode...

The page involves javascript: urls and (looks like) window.open.

It dies with:

Assertion: "bad mTransferAmount in READING" (mTransferAmount >= 0) at file
nsFileTransport.cpp, line 756
Break: at file nsFileTransport.cpp, line 756
Assertion failure: me == CurrentThreadId(), at jslock.c:715

Hm.  Not sure who to reassign it to...  CCed folks?  Any ideas?

Shawn: you can help us fix this, by boiling the popup script down to the
simplist possible page that still reproduces the bug.

Comment 2

[Brendan Eich [:brendan]]

Adding pollmann -- is this a popup DOM bug that's been fixed recently, or being
worked on presently?

/be

Comment 3

[Eric Pollmann]

Nothing on the stack I've seen before, sorry.

Comment 4

[Eric Pollmann]

Attachment: Reduced test case

Comment 5

[Eric Pollmann]

The simplest test case is simply to type this into the URL bar and press Enter:

javascript:window.open()

Comment 6

[Mike McCabe]

Brendan -

We need to get a good owner for this.  Who owns javascript: URLs these days?

Comment 7

[Brendan Eich [:brendan]]

I'm not sure this is a javascript: URL bug, because it should have crashed on
debug builds with an assertion if it were.  Nevertheless, I'm marking a dup of
1646.  See the bug that bug depends on, which dougt is about to fix, for more.

/be

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

Comment 8

[Joseph Elwell]

[bugday] marking verified. the url has a javascript url, that causes crash. And
just in case anyone doubts me. The WinNT build 1999121608 works fine on the page
now.