1277544 - Fix and enable test_blank_named_window.html, test_named_window.html and test_storage_copied.html tests for Android

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

Bug 1276738 added these tests, and then they got backed out because of Android failures.

I'm re-landing the tests with Android disabled, but we should really try to figure out why these are failing and fix them / re-enable them if possible.

Failure log sample:

200 INFO TEST-UNEXPECTED-FAIL | embedding/components/windowwatcher/test/test_blank_named_window.html | [Exception... "Failure" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: http://mochi.test:8888/tests/embedding/components/windowwatcher/test/test_blank_named_window.html :: <TOP_LEVEL> :: line 27" data: no]
204 INFO TEST-UNEXPECTED-FAIL | embedding/components/windowwatcher/test/test_named_window.html | [Exception... "Failure" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: http://mochi.test:8888/tests/embedding/components/windowwatcher/test/test_named_window.html :: <TOP_LEVEL> :: line 56" data: no]
208 INFO TEST-UNEXPECTED-FAIL | embedding/components/windowwatcher/test/test_storage_copied.html | [Exception... "Failure" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: http://mochi.test:8888/tests/embedding/components/windowwatcher/test/test_storage_copied.html :: <TOP_LEVEL> :: line 38" data: no]

Comment 1

[Mike Conley (:mconley) (:⚙️)]

So I talked to snorp about this.

All 3 of these tests work by setting browser.link.open_newwindow to 2, and then opening windows via window.open or <a target="_blank"> tags from web content.

According to snorp, we don't allow web content to open new top-level windows on Fennec - only privileged code can do that. So it is expected that browser.link.open_newwindow is throwing here when web content tries to open these windows.

So working as intended here. The tests should stay disabled as they do not apply to Fennec.