306630 - frame with src=javascript:... doesn't load.

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Johnny Stenback (:jst)]

After the split-window changes landed, a trivial frameset like this one doesn't
load the frame content:

<frameset rows="170,*">
<frame src="javascript:'frame1'">
<frame src="javascript:'frame2'">
</frameset>

Fix coming up.

Comment 1

[Johnny Stenback (:jst)]

Attachment: testcase

Comment 2

[Johnny Stenback (:jst)]

Attachment: Force creation of inner window when loading a javascript: URL into the window

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 194484 [details] [diff] [review]
Force creation of inner window when loading a javascript: URL into the window

r=mrbkap

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 194484 [details] [diff] [review]
Force creation of inner window when loading a javascript: URL into the window

sr=bzbarsky, but I'd almost rather have an EnsureInner() method on
nsPIDOMWindow that internally gets the document and that callers can use so
they don't have to depend on GetDocument creating the inner...	Perhaps for
1.9?

Comment 5

[Johnny Stenback (:jst)]

*** Bug 305656 has been marked as a duplicate of this bug. ***

Comment 6

[Brendan Eich [:brendan]]

Comment on attachment 194484 [details] [diff] [review]
Force creation of inner window when loading a javascript: URL into the window

Maybe ref this bug in the comment, so we can do what bz suggested, and just cuz
it's not totally clear why GetDocument creates a new inner?

a=me anyway.

/be

Comment 7

[Johnny Stenback (:jst)]

Done, and fixed on the trunk and branch. Filed bug 306642 on the issue bz
brought up...

Comment 8

[Boris Zbarsky [:bzbarsky]]

Added the testcase to reftest.