Revert of [Telemetry/Mac] Don't crash when setuid bit not set for powermetrics (patch...
commitfcc833eb2f5826f88685da6a6525a1237efd550c
authorrmcilroy <rmcilroy@chromium.org>
Wed, 15 Oct 2014 09:42:26 +0000 (15 02:42 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 15 Oct 2014 09:42:42 +0000 (15 09:42 +0000)
tree50e05ab1f828aa04f6e97c555ea9ab85debc16d6
parent8b146680e135820cccdd4c2fa5dce59fd1e84386
Revert of [Telemetry/Mac] Don't crash when setuid bit not set for powermetrics (patchset #1 id:1 of https://codereview.chromium.org/654803002/)

Reason for revert:
Broke the Mac perf bots: http://crbug.com/423688

BUG=423688

Original issue's description:
> [Telemetry/Mac] Don't crash when setuid bit not set for powermetrics
>
> When Telemetry is run it will prompt to set the setuid bit for powermetrics, if the user opts not to do this, Telemetry will crash because the Python call to terminate() doesn't have appropriate privileges to kill the process.
>
> The fix is to send SIGTERM via `sudo kill`.
>
> BUG=None
>
> Committed: https://crrev.com/abb19226cb71da65d798ef3d18527331bc349605
> Cr-Commit-Position: refs/heads/master@{#299506}

TBR=dtu@chromium.org,jeremy@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=None

Review URL: https://codereview.chromium.org/656723004

Cr-Commit-Position: refs/heads/master@{#299667}
tools/telemetry/telemetry/core/platform/power_monitor/powermetrics_power_monitor.py