Don't propagate synchronous Future.wait errors immediately.

tests/lib/async/future_test.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library future_test;
 
 import 'package:async_helper/async_helper.dart';
 import "package:expect/expect.dart";
 import 'dart:async';
 
@@ skipping 861 matching lines @@
     Expect.isTrue(index == 0 || index == 2, "$index");
     Expect.isFalse(uncaughts[index]);
     uncaughts[index] = true;
     asyncEnd();
   });
 }
 
 void testWaitSyncError() {
   var cms = const Duration(milliseconds: 100);
   var cleanups = new List.filled(3, false);
-  var uncaughts = new List.filled(3, false);
   asyncStart();
   asyncStart();
   runZoned(() {
     Future.wait(new Iterable.generate(5, (i) {
       if (i != 3) return new Future.delayed(cms * (i + 1), () => i);
       throw "throwing synchronously in iterable";
     }), cleanUp: (index) {
       Expect.isFalse(cleanups[index]);
       cleanups[index] = true;
       if (cleanups.every((x) => x)) asyncEnd();
     });
   }, onError: (e, s) {
     asyncEnd();
   });
 }
 
+void testWaitSyncError2() {
+  asyncStart();
+  Future.wait([null]).catchError((e, st) {
+    // Makes sure that the `catchError` is invoked.
+    // Regression test: an earlier version of `Future.wait` would propagate
+    // the error too soon for the code to install an error handler.
+    // `testWaitSyncError` didn't show this problem, because the `runZoned`
+    // was already installed.
+    asyncEnd();
+  });
+}
+
 void testBadFuture() {
   var bad = new BadFuture();
   // Completing with bad future (then call throws) puts error in result.
   asyncStart();
   Completer completer = new Completer();
   completer.complete(bad);
   completer.future.then((_) { Expect.fail("unreachable"); },
                         onError: (e, s) {
                           Expect.isTrue(completer.isCompleted);
                           asyncEnd();
@@ skipping 180 matching lines @@
 
   testChainedFutureValue();
   testChainedFutureValueDelay();
   testChainedFutureError();
 
   testSyncFuture_i13368();
 
   testWaitCleanUp();
   testWaitCleanUpError();
   testWaitSyncError();
+  testWaitSyncError2();
 
   testBadFuture();
 
   testTypes();
 
   testAnyValue();
   testAnyError();
   testAnyIgnoreIncomplete();
   testAnyIgnoreError();
 
@@ skipping 51 matching lines @@
   Future whenComplete(action()) {
     return new Future.microtask(action).then((_) => this);
   }
   Stream asStream() {
     return (new StreamController()..add(_result)..close()).stream;
   }
   Future timeout(Duration duration, {onTimeout()}) {
     return this;
   }
 }