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

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

Issue 10264005: Suppress get_drt/dartium download if an executable is provided via flag (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Added comment Created 8 years, 7 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 | « tools/testing/dart/drt_updater.dart ('k') | tools/testing/dart/test_suite.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 // 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:builtin"); 8 #import("dart:builtin");
9 #import("drt_updater.dart"); 9 #import("drt_updater.dart");
10 10
(...skipping 500 matching lines...) Expand 10 before | Expand all | Expand 10 after
511 if (configuration['compiler'].contains(',')) { 511 if (configuration['compiler'].contains(',')) {
512 return _expandHelper('compiler', configuration); 512 return _expandHelper('compiler', configuration);
513 } 513 }
514 514
515 // Expand runtimes. 515 // Expand runtimes.
516 var runtimes = configuration['runtime']; 516 var runtimes = configuration['runtime'];
517 if (runtimes.contains(',')) { 517 if (runtimes.contains(',')) {
518 return _expandHelper('runtime', configuration); 518 return _expandHelper('runtime', configuration);
519 } else { 519 } else {
520 // All runtimes eventually go through this path, after expansion. 520 // All runtimes eventually go through this path, after expansion.
521 var updater = runtimeUpdater(runtimes); 521 var updater = runtimeUpdater(configuration);
522 if (updater !== null) { 522 if (updater !== null) {
523 updater.update(); 523 updater.update();
524 } 524 }
525 } 525 }
526 526
527 // Adjust default timeout based on mode, compiler, and sometimes runtime. 527 // Adjust default timeout based on mode, compiler, and sometimes runtime.
528 if (configuration['timeout'] == -1) { 528 if (configuration['timeout'] == -1) {
529 var timeout = 60; 529 var timeout = 60;
530 switch (configuration['compiler']) { 530 switch (configuration['compiler']) {
531 case 'dartc': 531 case 'dartc':
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
628 return option; 628 return option;
629 } 629 }
630 } 630 }
631 print('Unknown test option $name'); 631 print('Unknown test option $name');
632 exit(1); 632 exit(1);
633 } 633 }
634 634
635 635
636 List<_TestOptionSpecification> _options; 636 List<_TestOptionSpecification> _options;
637 } 637 }
OLDNEW
« no previous file with comments | « tools/testing/dart/drt_updater.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698