Chromium Code Reviews| Index: tools/test-runtime.dart |
| diff --git a/tools/test-runtime.dart b/tools/test-runtime.dart |
| index 05aac408f812c5963e35404bcfcec82711c2c8c2..cb908280071c6ae69489aeda0adf3749e9cd7957 100755 |
| --- a/tools/test-runtime.dart |
| +++ b/tools/test-runtime.dart |
| @@ -65,18 +65,18 @@ main() { |
| ['Test configurations:'] : ['Test configuration:']; |
| for (Map conf in configurations) { |
| List settings = |
| - ['compiler', 'runtime', 'mode', 'arch'].map((name) => conf[name]); |
| + ['compiler', 'runtime', 'mode', 'arch'].mappedBy((name) => conf[name]); |
|
Bill Hesse
2013/01/09 17:06:33
You need toList here, unless you change settings t
kustermann
2013/01/09 18:02:25
Done.
|
| if (conf['checked']) settings.add('checked'); |
| output_words.add(Strings.join(settings, '_')); |
| } |
| print(Strings.join(output_words, ' ')); |
| } |
| - var configurationIterator = configurations.iterator(); |
| + var configurationIterator = configurations.iterator; |
| void enqueueConfiguration(ProcessQueue queue) { |
| - if (!configurationIterator.hasNext) return; |
| + if (!configurationIterator.moveNext()) return; |
| - var conf = configurationIterator.next(); |
| + var conf = configurationIterator.current; |
| if (selectors.containsKey('co19')) { |
| queue.addTestSuite(new Co19TestSuite(conf)); |
| } |
| @@ -100,7 +100,7 @@ main() { |
| // test that needs to load resources from the repo over http. |
| if (!listTests) { |
| // Only start the server if we are running browser tests. |
| - var runningBrowserTests = configurations.some((config) { |
| + var runningBrowserTests = configurations.any((config) { |
| return TestUtils.isBrowserRuntime(config['runtime']); |
| }); |
| if (runningBrowserTests) startHttpServer('127.0.0.1', 9876); |