Commit d90fefcf authored by timurrrr@chromium.org's avatar timurrrr@chromium.org

Fix my (??) stupid typo in memcheck_analyze which stupid python didn't caught.

As a result, memcheck_analyze was waiting for dead Valgrind processes for 3
extra minutes. D'oh.
BUG=17453,44545
TEST=trybots, experiments by eugenis@
Review URL: http://codereview.chromium.org/6258010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72119 0039d316-1c4b-4281-b951-d872f2087c98
parent 90b8ec57
...@@ -418,7 +418,7 @@ class MemcheckAnalyzer: ...@@ -418,7 +418,7 @@ class MemcheckAnalyzer:
# 3 minutes if it was killed. See http://crbug.com/17453 # 3 minutes if it was killed. See http://crbug.com/17453
ps_out = subprocess.Popen("ps p %s" % pid, shell=True, ps_out = subprocess.Popen("ps p %s" % pid, shell=True,
stdout=subprocess.PIPE).stdout stdout=subprocess.PIPE).stdout
if ps_out.readlines() < 2: if len(ps_out.readlines()) < 2:
running = False running = False
found = find_and_truncate(f) found = find_and_truncate(f)
if not running and not found: if not running and not found:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment