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

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: kershaw, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
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
(Reporter)

Updated

4 years ago
Blocks: 1110650
(Reporter)

Comment 1

4 years ago
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?
Flags: needinfo?(kchen)
(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?
Flags: needinfo?(kchen)
(Reporter)

Comment 3

4 years ago
I am not sure about the number.
Let me collect this kind of test cases first.
(Reporter)

Comment 4

4 years ago
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
(Reporter)

Comment 5

2 years ago
Close this bug since we don't support nested-oop now.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.