Open Bug 1403689 Opened 7 years ago Updated 1 year ago

Allow to pass a PID to Marionette client if it doesn't manage the instance itself

Categories

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

57 Branch
enhancement

Tracking

(Not tracked)

People

(Reporter: whimboo, Unassigned)

References

(Depends on 1 open bug)

Details

(Keywords: pi-marionette-client)

Even if Marionette doesn't manage the instance itself, it will be helpful to have a PID available of the process. Here some examples:

* A call to `wait_for_port` / `raise_for_port` could return early and not after 120s by default, if the process is not running anymore (maybe due to a crash)

* `_handle_socket_failure` could use the pid to at least check if the process is running, and could throw an appropriate exception
I don’t think we should pass a PID to the WebDriver:NewSession
command because the response from session creation will always
include the accurate PID of the current process in the moz:processID
field.
FYI this is for the Marionette Python client and not the server component. And we would need the PID before `start_session` has been called.
(In reply to Henrik Skupin (:whimboo) from comment #2)

> FYI this is for the Marionette Python client and not the server
> component.  And we would need the PID before `start_session` has
> been called.

Sorry, that wasn’t clear when you talked about “Marionette”.
Things are obviously different for the client because the lifetime
of the Firefox process is managed externally by a FirefoxRunner
instance.  My comment above is irrelevant to this end.
Summary: Allow to pass a PID to Marionette if it doesn't manage the instance itself → Allow to pass a PID to Marionette client if it doesn't manage the instance itself
Depends on: 1420372
[Mass Change 2018-01-15] Moving bugs to backlog
Priority: -- → P3
Severity: normal → S3
Product: Testing → Remote Protocol
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.