Closed Bug 929090 Opened 7 years ago Closed 5 years ago

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

Categories

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

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: ahal, Unassigned)

References

Details

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?
See Also: → 927586
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).
Do we know why we're calling open() using a window which doesn't have presentation anymore or yet?
(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,
Blocks: 931116
No longer blocks: 916127
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.