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

Side by Side Diff: tools/testing/dart/test_options.dart

Issue 11229067: Increase timeout to see if "timeout" errors on Windows bot goes away. Otherwise, (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 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 | Annotate | Revision Log
« 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 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #library("test_options_parser"); 5 #library("test_options_parser");
6 6
7 #import("dart:io"); 7 #import("dart:io");
8 #import("dart:math"); 8 #import("dart:math");
9 #import("drt_updater.dart"); 9 #import("drt_updater.dart");
10 #import("test_suite.dart"); 10 #import("test_suite.dart");
(...skipping 545 matching lines...) Expand 10 before | Expand all | Expand 10 after
556 } 556 }
557 if (configuration['host_checked']) { 557 if (configuration['host_checked']) {
558 timeout *= 16; 558 timeout *= 16;
559 } 559 }
560 if (configuration['checked']) { 560 if (configuration['checked']) {
561 timeout *= 2; 561 timeout *= 2;
562 } 562 }
563 if (Contains(configuration['runtime'], 563 if (Contains(configuration['runtime'],
564 const ['ie9', 'ie10', 'ff', 'chrome', 'safari', 564 const ['ie9', 'ie10', 'ff', 'chrome', 'safari',
565 'opera'])) { 565 'opera'])) {
566 timeout *= 8; // Allow additional time for browser testing to run. 566 // TODO(ahe): Restore the timeout to 8 times the original timeout
Jennifer Messerly 2012/10/23 21:05:55 might want to leave this TODO(efortuna)? (not that
567 // when dart2js compile-time performance has improved.
568 timeout *= 12; // Allow additional time for browser testing to run.
567 } 569 }
568 break; 570 break;
569 default: 571 default:
570 if (configuration['mode'] == 'debug') { 572 if (configuration['mode'] == 'debug') {
571 timeout *= 2; 573 timeout *= 2;
572 } 574 }
573 if (Contains(configuration['runtime'], const ['drt', 'dartium'])) { 575 if (Contains(configuration['runtime'], const ['drt', 'dartium'])) {
574 timeout *= 4; // Allow additional time for browser testing to run. 576 timeout *= 4; // Allow additional time for browser testing to run.
575 } 577 }
576 break; 578 break;
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
649 return option; 651 return option;
650 } 652 }
651 } 653 }
652 print('Unknown test option $name'); 654 print('Unknown test option $name');
653 exit(1); 655 exit(1);
654 } 656 }
655 657
656 658
657 List<_TestOptionSpecification> _options; 659 List<_TestOptionSpecification> _options;
658 } 660 }
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