[Telemetry] Move listing stray processes to more appropriate places

[Telemetry] Move listing stray processes to more appropriate places.

We move telemetry's stray processes listing registration to telemetry's
running entry points, i.e: benchmark_runner & run_tests instead of
telemetry/__init__.py to avoid pylint script picking up this registration
hook.

BUG=536862
Committed: https://crrev.com/77694fdaad974a760189b3e345fcd5486b15b022
Cr-Commit-Position: refs/heads/master@{#352757}

[nednguyen, 2015-10-06 03:38:37]

nednguyen@google.com changed reviewers:
+ dtu@chromium.org, jbudorick@chromium.org, kbr@chromium.org

[Ken Russell (switch to Gerrit), 2015-10-06 23:51:11]

lgtm

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/benchmark_runner.py (right):

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/benchmark_runner.py:391:
ps_util.EnablelListingStrayProcessesUponExitHook()
Typo. Enablel -> Enable

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/internal/util/ps_util.py (right):

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/internal/util/ps_util.py:54: def
EnablelListingStrayProcessesUponExitHook():
Same typo here.

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/testing/run_tests.py (right):

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/testing/run_tests.py:219:
ps_util.EnablelListingStrayProcessesUponExitHook()
Here too.

[nednguyen, 2015-10-07 00:22:44]

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/benchmark_runner.py (right):

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/benchmark_runner.py:391:
ps_util.EnablelListingStrayProcessesUponExitHook()
On 2015/10/06 23:51:10, Ken Russell wrote:
> Typo. Enablel -> Enable

Done.

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/internal/util/ps_util.py (right):

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/internal/util/ps_util.py:54: def
EnablelListingStrayProcessesUponExitHook():
On 2015/10/06 23:51:10, Ken Russell wrote:
> Same typo here.

Done.

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
File tools/telemetry/telemetry/testing/run_tests.py (right):

https://codereview.chromium.org/1379063003/diff/20001/tools/telemetry/telemet...
tools/telemetry/telemetry/testing/run_tests.py:219:
ps_util.EnablelListingStrayProcessesUponExitHook()
On 2015/10/06 23:51:10, Ken Russell wrote:
> Here too.

Done.

[nednguyen, 2015-10-07 00:23:07]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2015-10-07 00:23:08]

The patchset sent to the CQ was uploaded after l-g-t-m from kbr@chromium.org
Link to the patchset: https://codereview.chromium.org/1379063003/#ps40001
(title: "EnablelListingStrayProcessesUponExitHook -> EnableListing..")

[commit-bot: I haz the power, 2015-10-07 00:25:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1379063003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1379063003/40001

[commit-bot: I haz the power, 2015-10-07 02:06:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-07 02:06:51]

Failed to apply patch for tools/telemetry/telemetry/testing/run_tests.py:
While running git apply --index -3 -p1;
  error: patch failed: tools/telemetry/telemetry/testing/run_tests.py:215
  Falling back to three-way merge...
  Applied patch to 'tools/telemetry/telemetry/testing/run_tests.py' with
conflicts.
  U tools/telemetry/telemetry/testing/run_tests.py

Patch:       tools/telemetry/telemetry/testing/run_tests.py
Index: tools/telemetry/telemetry/testing/run_tests.py
diff --git a/tools/telemetry/telemetry/testing/run_tests.py
b/tools/telemetry/telemetry/testing/run_tests.py
index
15c7c9bb9b6eeadc58a3777d738ffd1172f79b55..306f246cf7186af138cdc8f4060295220312ef5f
100644
--- a/tools/telemetry/telemetry/testing/run_tests.py
+++ b/tools/telemetry/telemetry/testing/run_tests.py
@@ -13,6 +13,7 @@ from telemetry.internal.browser import browser_options
 from telemetry.internal.platform import device_finder
 from telemetry.internal.util import binary_manager
 from telemetry.internal.util import command_line
+from telemetry.internal.util import ps_util
 from telemetry.testing import browser_test_case
 from telemetry.testing import options_for_unittests
 
@@ -215,6 +216,7 @@ def _SetUpProcess(child, context): # pylint: disable=W0613
       level=logging.INFO,
       format='(%(levelname)s) %(asctime)s %(module)s.%(funcName)s:%(lineno)d  '
              '%(message)s')
+  ps_util.EnableListingStrayProcessesUponExitHook()
   args = context
   if args.device and args.device == 'android':
     android_devices = device_finder.GetDevicesMatchingOptions(args)

[commit-bot: I haz the power, 2015-10-07 02:08:24]

The CQ bit was unchecked by commit-bot@chromium.org

[nednguyen, 2015-10-07 02:12:18]

The CQ bit was checked by nednguyen@google.com

[nednguyen, 2015-10-07 02:12:19]

The patchset sent to the CQ was uploaded after l-g-t-m from kbr@chromium.org
Link to the patchset: https://codereview.chromium.org/1379063003/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2015-10-07 02:14:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1379063003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1379063003/60001

[commit-bot: I haz the power, 2015-10-07 02:37:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-10-07 02:37:27]

Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[nednguyen, 2015-10-07 02:39:31]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2015-10-07 02:40:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1379063003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1379063003/60001

[commit-bot: I haz the power, 2015-10-07 03:39:56]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-07 03:40:42]

Patchset 4 (id:??) landed as
https://crrev.com/77694fdaad974a760189b3e345fcd5486b15b022
Cr-Commit-Position: refs/heads/master@{#352757}