Closed Bug 1491839 Opened 6 years ago Closed 1 year ago

Remove start-up check for e10s status

Categories

(Testing :: Marionette Client and Harness, enhancement, P3)

Product:
Testing
Component:
Marionette Client and Harness
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: whimboo, Unassigned)

Details

Marionette client currently verifies the e10s status of Firefox. For that it has to create a separate session which immediately gets shutdown. 

https://treeherder.mozilla.org/logviewer.html#?job_id=199639409&repo=mozilla-central&lineNumber=986-1001

Here the related code:

https://dxr.mozilla.org/mozilla-central/rev/7ed950e60f3c1f8a47c117c04124d31e94a66e32/testing/marionette/harness/marionette_harness/runner/base.py#566-575

https://dxr.mozilla.org/mozilla-central/rev/7ed950e60f3c1f8a47c117c04124d31e94a66e32/testing/marionette/harness/marionette_harness/runner/base.py#837-850

We added that step when e10s was introduced to Firefox to ensure that we run the right mode. But that is not that important anymore now given that it is always enabled (except on Android). Also checking other test harnesses none of them actually do that, but only set the appropriate preference.

We should remove this extra step.
Severity: normal → S3
Product: Testing → Remote Protocol

This code is no longer present.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → INVALID
Component: Marionette → Marionette Client and Harness
Product: Remote Protocol → Testing
You need to log in before you can comment on or make changes to this bug.