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

Side by Side Diff: dart/tools/test.dart

Issue 23830002: Add again the *necessary* timeout markers for tests running on ie9/ie10 (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 3 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 | « dart/tests/lib/lib.status ('k') | 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 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 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 print('Started HttpServers: $commandline'); 152 print('Started HttpServers: $commandline');
153 }); 153 });
154 } 154 }
155 } 155 }
156 156
157 if (conf['runtime'].startsWith('ie')) { 157 if (conf['runtime'].startsWith('ie')) {
158 // NOTE: We've experienced random timeouts of tests on ie9/ie10. The 158 // NOTE: We've experienced random timeouts of tests on ie9/ie10. The
159 // underlying issue has not been determined yet. Our current hypothesis 159 // underlying issue has not been determined yet. Our current hypothesis
160 // is that windows makes bad scheduling decisions if we overload a 160 // is that windows makes bad scheduling decisions if we overload a
161 // machine (i.e. some processes seem to starve). 161 // machine (i.e. some processes seem to starve).
162 maxProcesses = math.max(1, maxProcesses-2);
163 maxBrowserProcesses = 1; 162 maxBrowserProcesses = 1;
164 } else if (conf['runtime'].startsWith('safari') && 163 } else if (conf['runtime'].startsWith('safari') &&
165 conf['use_browser_controller']) { 164 conf['use_browser_controller']) {
166 // Safari does not allow us to run from a fresh profile, so we can only 165 // Safari does not allow us to run from a fresh profile, so we can only
167 // use one browser. 166 // use one browser.
168 maxBrowserProcesses = 1; 167 maxBrowserProcesses = 1;
169 } 168 }
170 169
171 for (String key in selectors.keys) { 170 for (String key in selectors.keys) {
172 if (key == 'co19') { 171 if (key == 'co19') {
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
301 void main() { 300 void main() {
302 deleteTemporaryDartDirectories().then((_) { 301 deleteTemporaryDartDirectories().then((_) {
303 var optionsParser = new TestOptionsParser(); 302 var optionsParser = new TestOptionsParser();
304 var configurations = optionsParser.parse(new Options().arguments); 303 var configurations = optionsParser.parse(new Options().arguments);
305 if (configurations != null && configurations.length > 0) { 304 if (configurations != null && configurations.length > 0) {
306 testConfigurations(configurations); 305 testConfigurations(configurations);
307 } 306 }
308 }); 307 });
309 } 308 }
310 309
OLDNEW
« no previous file with comments | « dart/tests/lib/lib.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698