1118088 - [nested-oop] window.open() fails in nested content process

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Kershaw Chang [:kershaw]]

Found this bug when running test_other_auxiliary_navigation_by_location.html mochietst.

When running window.open("file_other_auxiliary_navigation_by_location.html", "otherWindow"), it fails in nsGlobalWindow::OpenInternal [1].

[1] http://dxr.mozilla.org/mozilla-central/source/dom/base/nsGlobalWindow.cpp#11814

Comment 1

[Kershaw Chang [:kershaw]]

It seems like that no one handles mozbrowseropenwindow event in grandchild process's parent.
Maybe we should disable test cases that calling window.open() for nested oop.

@Kan-Ru,
What do you think?

Comment 2

[Kan-Ru Chen [:kanru] (UTC+9)]

(In reply to Kershaw Chang [:kershaw] from comment #1)
> It seems like that no one handles mozbrowseropenwindow event in grandchild
> process's parent.
> Maybe we should disable test cases that calling window.open() for nested oop.
> 
> @Kan-Ru,
> What do you think?

Yes, disable them first. How many tests are affected?

Comment 3

[Kershaw Chang [:kershaw]]

I am not sure about the number.
Let me collect this kind of test cases first.

Comment 4

[Kershaw Chang [:kershaw]]

Currently, I found the following tests are affected. There could be more.

docshell/test/iframesandbox/test_other_auxiliary_navigation_by_location.html
docshell/test/iframesandbox/test_our_auxiliary_navigation_by_location.html
docshell/test/iframesandbox/test_top_navigation_by_location.html
docshell/test/iframesandbox/test_top_navigation_by_location_exotic.html
docshell/test/navigation/test_bug278916.html
docshell/test/navigation/test_bug279495.html
docshell/test/navigation/test_bug386782.html
docshell/test/navigation/test_bug430723.html

Comment 5

[Kershaw Chang [:kershaw]]

Close this bug since we don't support nested-oop now.