Chromium Code Reviews| Index: tools/testing/perf_testing/run_perf_tests.py |
| =================================================================== |
| --- tools/testing/perf_testing/run_perf_tests.py (revision 6914) |
| +++ tools/testing/perf_testing/run_perf_tests.py (working copy) |
| @@ -401,7 +401,7 @@ |
| '--dartium'] |
| # TODO(vsm): It's inconvenient that run_cmd isn't in scope here. |
| # Perhaps there is a better place to put that or this. |
| - subprocess.Popen(get_dartium, shell=has_shell) |
| + subprocess.call(get_dartium, shell=has_shell) |
|
Emily Fortuna
2012/04/25 18:38:58
How about:
p = subprocess.Popen(cmd_list, stdout=s
|
| return browsers |
| @@ -617,7 +617,7 @@ |
| self.test.test_runner.run_cmd( |
| ['python', os.path.join('tools', 'testing', 'run_selenium.py'), |
| '--out', file_path, '--browser', browser, |
| - '--timeout', '600', '--mode', 'dromaeo'], self.test.trace_file, |
| + '--timeout', '900', '--mode', 'dromaeo'], self.test.trace_file, |
| append=True) |
| @staticmethod |