diff options
author | v.putkinen <v.putkinen@partner.samsung.com> | 2014-09-23 07:29:26 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-23 14:29:55 +0000 |
commit | 362315d0dee01153479d649db12e1e6edd2622af (patch) | |
tree | 0f62816efa0ef98ded0ba978a5068990dff97f19 /mojo/build | |
parent | d1f5016cb601daa67da3ca36eff3c8af1f35fa60 (diff) | |
download | chromium_src-362315d0dee01153479d649db12e1e6edd2622af.zip chromium_src-362315d0dee01153479d649db12e1e6edd2622af.tar.gz chromium_src-362315d0dee01153479d649db12e1e6edd2622af.tar.bz2 |
telemetry: Fix runaway subprocess in monsoon_power_monitor.py
Telemetry leaves a memory hogging subprocess behind if:
1) A Monsoon Power Monitor is attached to host computer (Linux),
2) User runs a power-metric-using benchmark such as sunspider on Android
ChromeShell, and
3) Benchmark raises an exception between power metric's Start and Stop
calls. For example, an exception will occur if browser crashes and
the benchmark does something like tab.WaitForJavaScriptExpression().
This patch fixes the issue by setting the daemon flag on the subprocess.
This will cause the main process to kill the subprocess on exit.
BUG=413596
R=ernstm@chromium.org
Review URL: https://codereview.chromium.org/572643002
Cr-Commit-Position: refs/heads/master@{#296177}
Diffstat (limited to 'mojo/build')
0 files changed, 0 insertions, 0 deletions