B2G desktop mochitest uncaught exception (0x80004005) at [nsIDOMJSWindow.open]

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
5 years ago
3 years ago

People

(Reporter: ahal, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=29444607&tree=Cedar

The tests that failed due to bug 927586 now have this failure message (they all call window.open()). Note in bug 927586, we learned that a call to nsDOMWindowUtils::GetPresContext() was returning null.

When running locally, they still fail, but with a different error:
https://pastebin.mozilla.org/3306936

For the record, the known tests exhibiting this failure (though there will be others) are:
content/base/test/test_bug326337.html
content/base/test/test_bug426646.html
content/base/test/test_bug557892.html
content/events/test/test_bug322588.html

Should these tests even be running on B2G desktop? Is it possible to get this fixed?
(Reporter)

Updated

5 years ago
See Also: → bug 927586
(Reporter)

Comment 1

5 years ago
STR:
./mach mochitest-b2g-desktop

Note: because of bug 927889, specifying a single test file to the mach command doesn't work on b2g desktop. If you want to run a single test, edit testing/mochitest/b2g.json and comment out everything under 'runtests', then add the full paths to the test files you want to run (e.g content/base/test/test_bug326337.html).

Comment 2

5 years ago
Do we know why we're calling open() using a window which doesn't have presentation anymore or yet?
(Reporter)

Comment 3

5 years ago
(In reply to Olli Pettay [:smaug] from comment #2)
> Do we know why we're calling open() using a window which doesn't have
> presentation anymore or yet?

I'm not sure. all I can say is these tests pass both on desktop firefox and b2g emulators. I think we'll need someone more familiar with b2g desktop builds to answer that.

Though it's quite possible the way forward is to either disable these tests on b2g desktop or modify the tests to ensure we have a prescontext before calling window.open. If we did this it would probably be on a test by test basis.
Jonas and I spoke about the issue of "b2g desktop ownership" earlier.  I'm going to see if I can get some clarity of ownership as I think it'll help with bugs like these,
(Reporter)

Updated

5 years ago
Blocks: 931116
(Reporter)

Updated

5 years ago
No longer blocks: 916127
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.