Get mochitests in Gaia OOP mode running on Mulet

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: jgriffin, Assigned: gerard-majax)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Our current mochitests on Mulet are run in-process, just like desktop Firefox.  We also want to run them out-of-process, in a Gaia app, just like we do for b2gdesktop.

This almost works right now, but the tests still run in the main screen after generating this error:

JavascriptException: JavascriptException: TypeError: homescreen is null
stacktrace:
	execute_script @runtestsb2g.py, line 290
	inline javascript, line 22
	src: "let container = homescreen.contentWindow.document.getElementById('test-container');"

I think it's likely a timing issue.
(Assignee)

Comment 2

3 years ago
Created attachment 8665717 [details] [diff] [review]
Bug XXX - Running Mochitest OOP on Mulet r=...

That seems to be running ?
Attachment #8665717 - Flags: feedback?(jgriffin)
(Assignee)

Updated

3 years ago
Assignee: nobody → lissyx+mozillians
(Reporter)

Comment 3

3 years ago
What's currently running is the same as mochitest on (non-e10s) desktop Firefox.  What used to be running on b2gdesktop was mochitest inside a Gaia OOP test container app.  I'm not sure whether that's still needed or not.
(Reporter)

Comment 4

3 years ago
Comment on attachment 8665717 [details] [diff] [review]
Bug XXX - Running Mochitest OOP on Mulet r=...

Review of attachment 8665717 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.

::: b2g/app/b2g.js
@@ +296,5 @@
>  // Layers Acceleration.  We can only have nice things on gonk, because
>  // they're not maintained anywhere else.
>  pref("layers.offmainthreadcomposition.enabled", true);
>  #ifndef MOZ_WIDGET_GONK
> +pref("dom.ipc.tabs.disabled", false);

This is already set to false when --oop at https://dxr.mozilla.org/mozilla-central/source/b2g/components/OopCommandLine.js#42; is this still needed?
Attachment #8665717 - Flags: feedback?(jgriffin) → feedback+
(Assignee)

Updated

2 years ago
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.