Isolate.spawn{Uri} only reports errors asynchronously.

runtime/lib/schedule_microtask_patch.dart

Index: runtime/lib/schedule_microtask_patch.dart
diff --git a/runtime/lib/schedule_microtask_patch.dart b/runtime/lib/schedule_microtask_patch.dart
index 75e87c8eb2fde52d0e0fa2e426e0a8478f0f2960..62d1184911a669732deb566831f983b20c765bcf 100644
--- a/runtime/lib/schedule_microtask_patch.dart
+++ b/runtime/lib/schedule_microtask_patch.dart
@@ -2,24 +2,9 @@
 // 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.
 
-patch class _AsyncRun {
-  /* patch */ static void _scheduleImmediate(void callback()) {
-    if (_ScheduleImmediate._closure == null) {
-      // TODO(9001): don't default to using the Timer to enqueue the immediate
-      //             callback.
-      _createTimer(Duration.ZERO, callback);
-      return;
-    }
-    _ScheduleImmediate._closure(callback);
-  }
-}
-
-typedef void _ScheduleImmediateClosure(void callback());
-
+@deprecated

[Lasse Reichstein Nielsen, 2013/11/27 11:41:28]

Why deprecate this and add another?
Can we just keep this one?

[floitsch, 2013/11/27 13:57:06]

No.
This one is not in _collection-dev.
The only reason it is still here is, to have a smooth transition for the Dartium
bindings that set this closure instead of setting the one in _collection-dev.
I have already a patch for it, and we can then remove this one.
https://chromiumcodereview.appspot.com/85333004/

 class _ScheduleImmediate {
-  static _ScheduleImmediateClosure _closure;
-}
-
-void _setScheduleImmediateClosure(_ScheduleImmediateClosure closure) {
-  _ScheduleImmediate._closure = closure;
+  static set _closure(closure) {
+    scheduleImmediateClosure = closure;
+  }
 }