Set marionette.actors.enabled to true by default for Fission builds
Categories
(Remote Protocol :: Marionette, task, P1)
Tracking
(Fission Milestone:M7, firefox84 fixed)
Tracking | Status | |
---|---|---|
firefox84 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
(Whiteboard: [marionette-fission-mvp][simple])
Attachments
(1 file)
Bug 1660168 - [marionette] Enable usage of Marionette's JSWindowActor by default for Fission builds.
47 bytes,
text/x-phabricator-request
|
Details | Review |
Once we have an actor code path for all the commands we should enable the preference by default for Fission builds in CI. That should happen for the Mn, Wdspec, and web-platform tests including reftests.
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Moving marionette-fission-mvp bugs from Fission Nightly Experiment milestone (M6b) to Fission Beta milestone (M7).
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Here a try build for Fission related jobs using Marionette / geckodriver:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2f55fe8edaa1d7f292df86060fc9ae81d04421f5
I will add blocking bugs to the dependency list as soon as test results are available.
Assignee | ||
Comment 4•4 years ago
|
||
With the duplicate bug 1673326 it's clear that we also need a way to allow tests to query for the actor state. Right now we only set the internal state but don't update the preference. Also we run the preference check each and every time a command gets called. We should change it so that it's done only once for the very first time.
Assignee | ||
Comment 5•4 years ago
|
||
Most recent try build with the patch from the last remaining bug included is totally green:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6d0386b679d693edaa5ebe1e2e63279db4befc13
So lets get this enabled tomorrow.
Assignee | ||
Comment 6•4 years ago
|
||
Comment 8•4 years ago
|
||
bugherder |
Updated•2 years ago
|
Description
•