912285 - [mozprocess] ProcessHandler should front-end more subprocess API

Status: RESOLVED WONTFIX

(Testing :: Mozbase, defect)

Description

[Jeff Hammel]

Functionality needed for mochitest runtests.py.  See
https://bugzilla.mozilla.org/show_bug.cgi?id=746243

- [mozprocess] ProcessHandler.run()
  should return the PID
  https://github.com/mozilla/mozbase/blob/master/mozprocess/mozprocess/processhandler.py#L535

- [mozprocess] self.pid should return 'None' before process start and
  after process finish

- [mozprocess] ProcessHandler should add .poll(),
  .terminate() (and/or .send_signal())
  http://mxr.mozilla.org/mozilla-central/source/testing/mochitest/runtests.py#114
  http://mxr.mozilla.org/mozilla-central/source/testing/mochitest/runtests.py#116

Note that reimplementing ProcessHandler as a subclass of
subprocess.Popen -- bug 819547 -- takes care of the above

Comment 1

[Jeff Hammel]

The blocker is loose as it can be worked around.  If desired, I can break this up into multiple bugs, but the overall code change will be small.

Comment 2

[Jeff Hammel]

Note also that if we need legacy compatability (which we will) we can preserve a wrapper around the new API complete wth deprecation warnings.

Comment 3

[Geoff Brown [:gbrown]]

Mass closing bugs with no activity in 2+ years. If this bug is important to you, please re-open.