Add chain support to barback.

sdk/lib/_internal/pub/lib/src/barback/load_transformers.dart

 // Copyright (c) 2013, 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 pub.load_transformers;
 
 import 'dart:async';
 import 'dart:convert';
 import 'dart:isolate';
 
@@ skipping 277 matching lines @@
   /// The name of the type of exception thrown.
   ///
   /// This is the return value of [error.runtimeType.toString()]. Keep in mind
   /// that objects in different libraries may have the same type name.
   final String type;
 
   /// The exception's message, or its [toString] if it didn't expose a `message`
   /// property.
   final String message;
 
-  /// The exception's stack trace, or `null` if no stack trace was available.
-  final Trace stackTrace;
+  /// The exception's stack chain, or `null` if no stack chain was available.
+  final Chain stackTrace;
 
-  /// Loads a [CrossIsolateException] from a map.
+  /// Loads a [CrossIsolateException] from a serialized representation.
   ///
   /// [error] should be the result of [CrossIsolateException.serialize].
-  CrossIsolateException.deserialize(Map error)
-      : type = error['type'],
-        message = error['message'],
-        stackTrace = error['stack'] == null ? null :
-            new Trace.parse(error['stack']);
+  factory CrossIsolateException.deserialize(Map error) {
+    var type = error['type'];
+    var message = error['message'];
+    var stackTrace = error['stack'] == null ? null :
+            new Chain.parse(error['stack']);
+    return new CrossIsolateException._(type, message, stackTrace);
+  }
 
-  /// Serializes [error] to a map that can safely be passed across isolate
+  /// Loads a [CrossIsolateException] from a serialized representation.
+  ///
+  /// [error] should be the result of [CrossIsolateException.serialize].

[Bob Nystrom, 2013/12/04 17:37:21]

Update the doc comment.

[nweiz, 2013/12/04 22:41:30]

I've just gone back to not using a factory constructor; I'm not sure why I moved
away from one in the first place.

+  CrossIsolateException._(this.type, this.message, this.stackTrace);
+
+  /// Serializes [error] to an object that can safely be passed across isolate
   /// boundaries.
   static Map serialize(error, [StackTrace stack]) {
     if (stack == null && error is Error) stack = error.stackTrace;
     return {
       'type': error.runtimeType.toString(),
       'message': getErrorMessage(error),
-      'stack': stack == null ? null : stack.toString()
+      'stack': stack == null ? null : new Chain.forTrace(stack).toString()
     };
   }
 
   String toString() => "\$message\\n\$stackTrace";
 }
 
 // Get a string description of an exception.
 //
 // Most exception types have a "message" property. We prefer this since
 // it skips the "Exception:", "HttpException:", etc. prefix that calling
@@ skipping 237 matching lines @@
     'replyTo': receivePort.sendPort
   });
 
   return receivePort.first.then((response) {
     if (response['type'] == 'success') return response['value'];
     assert(response['type'] == 'error');
     return new Future.error(
         new dart.CrossIsolateException.deserialize(response['error']));
   });
 }