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

Side by Side Diff: tools/test.dart

Issue 17037003: Change number of started browser processes to the max number of browser processes. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | 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 dart 1 #!/usr/bin/env dart
2 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 2 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
3 // for details. All rights reserved. Use of this source code is governed by a 3 // for details. All rights reserved. Use of this source code is governed by a
4 // BSD-style license that can be found in the LICENSE file. 4 // BSD-style license that can be found in the LICENSE file.
5 5
6 /** 6 /**
7 * This file is the entrypoint of the dart test suite. This suite is used 7 * This file is the entrypoint of the dart test suite. This suite is used
8 * to test: 8 * to test:
9 * 9 *
10 * 1. the dart vm 10 * 1. the dart vm
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 // There should not be more than one InternetExplorerDriver instance 160 // There should not be more than one InternetExplorerDriver instance
161 // running at a time. For details, see 161 // running at a time. For details, see
162 // http://code.google.com/p/selenium/wiki/InternetExplorerDriver. 162 // http://code.google.com/p/selenium/wiki/InternetExplorerDriver.
163 if (conf['runtime'].startsWith('ie')) { 163 if (conf['runtime'].startsWith('ie')) {
164 maxBrowserProcesses = 1; 164 maxBrowserProcesses = 1;
165 } else if (conf['runtime'].startsWith('safari') && 165 } else if (conf['runtime'].startsWith('safari') &&
166 conf['use_browser_controller']) { 166 conf['use_browser_controller']) {
167 // FIXME(kustermann/ricow): Remove this once the new browser_controller is 167 // FIXME(kustermann/ricow): Remove this once the new browser_controller is
168 // stable. 168 // stable.
169 maxBrowserProcesses = math.max(maxProcesses - 3, 1); 169 maxBrowserProcesses = math.max(maxProcesses - 3, 1);
170 maxProcesses = math.max(maxProcesses -1, 1);
170 } 171 }
171 172
172 for (String key in selectors.keys) { 173 for (String key in selectors.keys) {
173 if (key == 'co19') { 174 if (key == 'co19') {
174 testSuites.add(new Co19TestSuite(conf)); 175 testSuites.add(new Co19TestSuite(conf));
175 } else if (conf['runtime'] == 'vm' && key == 'vm') { 176 } else if (conf['runtime'] == 'vm' && key == 'vm') {
176 // vm tests contain both cc tests (added here) and dart tests (added 177 // vm tests contain both cc tests (added here) and dart tests (added
177 // in [TEST_SUITE_DIRECTORIES]). 178 // in [TEST_SUITE_DIRECTORIES]).
178 testSuites.add(new VMTestSuite(conf)); 179 testSuites.add(new VMTestSuite(conf));
179 } else if (conf['analyzer']) { 180 } else if (conf['analyzer']) {
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 void main() { 309 void main() {
309 deleteTemporaryDartDirectories().then((_) { 310 deleteTemporaryDartDirectories().then((_) {
310 var optionsParser = new TestOptionsParser(); 311 var optionsParser = new TestOptionsParser();
311 var configurations = optionsParser.parse(new Options().arguments); 312 var configurations = optionsParser.parse(new Options().arguments);
312 if (configurations != null && configurations.length > 0) { 313 if (configurations != null && configurations.length > 0) {
313 testConfigurations(configurations); 314 testConfigurations(configurations);
314 } 315 }
315 }); 316 });
316 } 317 }
317 318
OLDNEW
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698