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

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

Issue 2879153005: Add support to dart2js for option --enable-asserts. (Closed)
Patch Set: Added !$checked to section predicate in co19 status file Created 3 years, 5 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
« no previous file with comments | « tools/testing/dart/options.dart ('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 // 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 import 'dart:async'; 5 import 'dart:async';
6 import 'dart:io'; 6 import 'dart:io';
7 import 'dart:math' as math; 7 import 'dart:math' as math;
8 8
9 import 'android.dart'; 9 import 'android.dart';
10 import 'browser_controller.dart'; 10 import 'browser_controller.dart';
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 for (var configuration in configurations) { 97 for (var configuration in configurations) {
98 var settings = [ 98 var settings = [
99 configuration.compiler.name, 99 configuration.compiler.name,
100 configuration.runtime.name, 100 configuration.runtime.name,
101 configuration.mode.name, 101 configuration.mode.name,
102 configuration.architecture.name 102 configuration.architecture.name
103 ]; 103 ];
104 if (configuration.isChecked) settings.add('checked'); 104 if (configuration.isChecked) settings.add('checked');
105 if (configuration.isStrong) settings.add('strong'); 105 if (configuration.isStrong) settings.add('strong');
106 if (configuration.useFastStartup) settings.add('fast-startup'); 106 if (configuration.useFastStartup) settings.add('fast-startup');
107 if (configuration.useEnableAsserts) settings.add('enable-asserts');
107 outputWords.add(settings.join('_')); 108 outputWords.add(settings.join('_'));
108 } 109 }
109 print(outputWords.join(' ')); 110 print(outputWords.join(' '));
110 } 111 }
111 112
112 var runningBrowserTests = 113 var runningBrowserTests =
113 configurations.any((config) => config.runtime.isBrowser); 114 configurations.any((config) => config.runtime.isBrowser);
114 115
115 var serverFutures = <Future>[]; 116 var serverFutures = <Future>[];
116 var testSuites = <TestSuite>[]; 117 var testSuites = <TestSuite>[];
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
259 // Start all the HTTP servers required before starting the process queue. 260 // Start all the HTTP servers required before starting the process queue.
260 if (!serverFutures.isEmpty) { 261 if (!serverFutures.isEmpty) {
261 await Future.wait(serverFutures); 262 await Future.wait(serverFutures);
262 } 263 }
263 264
264 // [firstConf] is needed here, since the ProcessQueue needs to know the 265 // [firstConf] is needed here, since the ProcessQueue needs to know the
265 // settings of 'noBatch' and 'local_ip' 266 // settings of 'noBatch' and 'local_ip'
266 new ProcessQueue(firstConf, maxProcesses, maxBrowserProcesses, startTime, 267 new ProcessQueue(firstConf, maxProcesses, maxBrowserProcesses, startTime,
267 testSuites, eventListener, allTestsFinished, verbose, adbDevicePool); 268 testSuites, eventListener, allTestsFinished, verbose, adbDevicePool);
268 } 269 }
OLDNEW
« no previous file with comments | « tools/testing/dart/options.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698