Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(152)

Side by Side Diff: tools/run-bisect-perf-regression.py

Issue 647163003: Change constructor call in run-bisect-perf-regression.py (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Run Performance Test Bisect Tool 6 """Run Performance Test Bisect Tool
7 7
8 This script is used by a try bot to run the bisect script with the parameters 8 This script is used by a try bot to run the bisect script with the parameters
9 specified in the bisect config file. It checks out a copy of the depot in 9 specified in the bisect config file. It checks out a copy of the depot in
10 a subdirectory 'bisect' of the working directory provided, annd runs the 10 a subdirectory 'bisect' of the working directory provided, annd runs the
(...skipping 227 matching lines...) Expand 10 before | Expand all | Expand 10 after
238 238
239 Attempts to build and run the current revision with and without the 239 Attempts to build and run the current revision with and without the
240 current patch, with the parameters passed in. 240 current patch, with the parameters passed in.
241 """ 241 """
242 # Bisect script expects to be run from the src directory 242 # Bisect script expects to be run from the src directory
243 os.chdir(SRC_DIR) 243 os.chdir(SRC_DIR)
244 244
245 bisect_utils.OutputAnnotationStepStart('Building With Patch') 245 bisect_utils.OutputAnnotationStepStart('Building With Patch')
246 246
247 opts = _CreateBisectOptionsFromConfig(config) 247 opts = _CreateBisectOptionsFromConfig(config)
248 b = bisect_perf_regression.BisectPerformanceMetrics(None, opts) 248 b = bisect_perf_regression.BisectPerformanceMetrics(opts)
249 249
250 if bisect_utils.RunGClient(['runhooks']): 250 if bisect_utils.RunGClient(['runhooks']):
251 raise RuntimeError('Failed to run gclient runhooks') 251 raise RuntimeError('Failed to run gclient runhooks')
252 252
253 if not b.BuildCurrentRevision('chromium'): 253 if not b.BuildCurrentRevision('chromium'):
254 raise RuntimeError('Patched version failed to build.') 254 raise RuntimeError('Patched version failed to build.')
255 255
256 bisect_utils.OutputAnnotationStepClosed() 256 bisect_utils.OutputAnnotationStepClosed()
257 bisect_utils.OutputAnnotationStepStart('Running With Patch') 257 bisect_utils.OutputAnnotationStepStart('Running With Patch')
258 258
(...skipping 298 matching lines...) Expand 10 before | Expand all | Expand 10 after
557 if config and config_is_valid: 557 if config and config_is_valid:
558 return _SetupAndRunPerformanceTest(config, opts.path_to_goma) 558 return _SetupAndRunPerformanceTest(config, opts.path_to_goma)
559 559
560 print ('Error: Could not load config file. Double check your changes to ' 560 print ('Error: Could not load config file. Double check your changes to '
561 'auto_bisect/bisect.cfg or run-perf-test.cfg for syntax errors.\n') 561 'auto_bisect/bisect.cfg or run-perf-test.cfg for syntax errors.\n')
562 return 1 562 return 1
563 563
564 564
565 if __name__ == '__main__': 565 if __name__ == '__main__':
566 sys.exit(main()) 566 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698