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

Side by Side Diff: scripts/slave/chromium/layout_test_wrapper.py

Issue 2384243005: Add initial CL for RandomOrder chromium.fyi bot (Closed)
Patch Set: Update build machine based on new bot allocation Created 4 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
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 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 """A wrapper script to run layout tests on the buildbots. 6 """A wrapper script to run layout tests on the buildbots.
7 7
8 Runs the run-webkit-tests script found in third_party/WebKit/Tools/Scripts above 8 Runs the run-webkit-tests script found in third_party/WebKit/Tools/Scripts above
9 this script. For a complete list of command-line options, pass '--help' on the 9 this script. For a complete list of command-line options, pass '--help' on the
10 command line. 10 command line.
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 if options.target: 88 if options.target:
89 command.extend(['--target', options.target]) 89 command.extend(['--target', options.target])
90 if options.platform: 90 if options.platform:
91 command.extend(['--platform', options.platform]) 91 command.extend(['--platform', options.platform])
92 if options.skipped: 92 if options.skipped:
93 command.extend(['--skipped', options.skipped]) 93 command.extend(['--skipped', options.skipped])
94 if options.no_pixel_tests: 94 if options.no_pixel_tests:
95 command.append('--no-pixel-tests') 95 command.append('--no-pixel-tests')
96 if options.enable_wptserve: 96 if options.enable_wptserve:
97 command.append('--enable-wptserve') 97 command.append('--enable-wptserve')
98 # TODO: Do we need to add --order and --seed here?
qyearsley 2016/10/07 21:56:58 Ah, good question -- I think we do. Note, I think
jeffcarp 2016/10/07 23:16:35 Added.
98 if options.batch_size: 99 if options.batch_size:
99 command.extend(['--batch-size', options.batch_size]) 100 command.extend(['--batch-size', options.batch_size])
100 if options.run_part: 101 if options.run_part:
101 command.extend(['--run-part', options.run_part]) 102 command.extend(['--run-part', options.run_part])
102 if options.builder_name: 103 if options.builder_name:
103 command.extend(['--builder-name', options.builder_name]) 104 command.extend(['--builder-name', options.builder_name])
104 if options.build_number: 105 if options.build_number:
105 command.extend(['--build-number', options.build_number]) 106 command.extend(['--build-number', options.build_number])
106 command.extend(['--master-name', slave_utils.GetActiveMaster() or '']) 107 command.extend(['--master-name', slave_utils.GetActiveMaster() or ''])
107 if options.step_name: 108 if options.step_name:
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
265 options, args = option_parser.parse_args() 266 options, args = option_parser.parse_args()
266 options.build_dir = build_directory.GetBuildOutputDirectory() 267 options.build_dir = build_directory.GetBuildOutputDirectory()
267 268
268 # Disable pageheap checking except on Windows. 269 # Disable pageheap checking except on Windows.
269 if sys.platform != 'win32': 270 if sys.platform != 'win32':
270 options.enable_pageheap = False 271 options.enable_pageheap = False
271 return layout_test(options, args) 272 return layout_test(options, args)
272 273
273 if '__main__' == __name__: 274 if '__main__' == __name__:
274 sys.exit(main()) 275 sys.exit(main())
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698