| commit | e5c729f794a24ba5195f8e56bd90a8b179097bdb | |
| author | skyostil <skyostil@chromium.org> | |
| Mon, 27 Jul 2015 17:36:38 +0000 (27 10:36 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Mon, 27 Jul 2015 17:37:30 +0000 (27 17:37 +0000) | ||
| tree | d523feceb4c5b51fb06e59f8e6411d255312a7a3 | treesnapshot (tar.gz zip) |
| parent | 477fc5da75c691f991eace64ad8877d7cef8aa95 | commitdiff |
telemetry: Fix killing the perf profiler
Telemetry was failing to kill the perf profiler on Android because
pylib's KillAll command didn't account for multiple instances of the
same binary being running. The KillAll function would randomly kill one
of the running instances and then time out waiting for the instances to
go away.
This patch changes pylib's KillAll helper send the signal to all running
instances instead of just one. To do that we need to extend the GetPids
function to return all running instances of a binary instead of just one.
BUG=512564
Review URL: https://codereview.chromium.org/1254843002
Cr-Commit-Position: refs/heads/master@{#340496}
Telemetry was failing to kill the perf profiler on Android because
pylib's KillAll command didn't account for multiple instances of the
same binary being running. The KillAll function would randomly kill one
of the running instances and then time out waiting for the instances to
go away.
This patch changes pylib's KillAll helper send the signal to all running
instances instead of just one. To do that we need to extend the GetPids
function to return all running instances of a binary instead of just one.
BUG=512564
Review URL: https://codereview.chromium.org/1254843002
Cr-Commit-Position: refs/heads/master@{#340496}