Properly parse jsshell and d8 stack frames.

pkg/stack_trace/lib/src/frame.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 frame;
 
 
 import 'package:path/path.dart' as path;
 
 import 'trace.dart';
@@ skipping 118 matching lines @@
 
   /// Parses a string representation of a Chrome/V8 stack frame.
   factory Frame.parseV8(String frame) {
     var match = _v8Frame.firstMatch(frame);
     if (match == null) {
       throw new FormatException("Couldn't parse V8 stack trace line '$frame'.");
     }
 
     // V8 stack frames can be in two forms.
     if (match[2] != null) {
-      // The first form looks like "  at FUNCTION (URI:LINE:COL)"
-      var uri = Uri.parse(match[2]);
+      // The first form looks like " at FUNCTION (PATH:LINE:COL)". PATH is
+      // usually an absolute URL, but it can be a path if the stack frame came
+      // from d8.
+      var uri = _uriOrPathToUri(match[2]);
       var member = match[1].replaceAll("<anonymous>", "<fn>");
       return new Frame(uri, int.parse(match[3]), int.parse(match[4]), member);
     } else {
-      // The second form looks like " at URI:LINE:COL", and is used for
-      // anonymous functions.
-      var uri = Uri.parse(match[5]);
+      // The second form looks like " at PATH:LINE:COL", and is used for
+      // anonymous functions. PATH is usually an absolute URL, but it can be a
+      // path if the stack frame came from d8.
+      var uri = _uriOrPathToUri(match[5]);
       return new Frame(uri, int.parse(match[6]), int.parse(match[7]), "<fn>");
     }
   }
 
   /// Parses a string representation of an IE stack frame.
   ///
   /// IE10+ frames look just like V8 frames. Prior to IE10, stack traces can't
   /// be retrieved.
   factory Frame.parseIE(String frame) => new Frame.parseV8(frame);
 
   /// Parses a string representation of a Firefox stack frame.
   factory Frame.parseFirefox(String frame) {
     var match = _firefoxFrame.firstMatch(frame);
     if (match == null) {
       throw new FormatException(
           "Couldn't parse Firefox stack trace line '$frame'.");
     }
 
-    var uri = Uri.parse(match[3]);
+    // Normally this is a URI, but in a jsshell trace it can be a path.
+    var uri = _uriOrPathToUri(match[3]);
     var member = match[1];
     member += new List.filled('/'.allMatches(match[2]).length, ".<fn>").join();
     if (member == '') member = '<fn>';
 
     // Some Firefox members have initial dots. We remove them for consistency
     // with other platforms.
     member = member.replaceFirst(_initialDot, '');
     return new Frame(uri, int.parse(match[4]), null, member);
   }
 
@@ skipping 32 matching lines @@
     // relative to the current working directory.
     if (uri.scheme == '') {
       uri = path.toUri(path.absolute(path.fromUri(uri)));
     }
 
     var line = match[2] == null ? null : int.parse(match[2]);
     var column = match[3] == null ? null : int.parse(match[3]);
     return new Frame(uri, line, column, match[4]);
   }
 
+  /// A regular expression matching an absolute URI.
+  static final _uriRegExp = new RegExp(r'^[a-zA-Z][-+.a-zA-Z\d]*://');
+
+  /// A regular expression matching a Windows path.
+  static final _windowsRegExp = new RegExp(r'^[a-zA-Z]:[\/]');

[Bob Nystrom, 2013/11/07 22:25:10]

What about UNC paths?

[nweiz, 2013/11/07 22:40:12]

They were implicitly supported by the uriOrPath.contains('\') case, but I've
added more explicit support.

+
+  /// Converts [uriOrPath], which can be a URI, a Windows path, or a Posix path,
+  /// to a URI (absolute if possible).
+  static Uri _uriOrPathToUri(String uriOrPath) {
+    if (uriOrPath.contains(_uriRegExp)) {
+      return Uri.parse(uriOrPath);
+    } else if (uriOrPath.contains(_windowsRegExp)) {
+      return new Uri.file(uriOrPath, windows: true);
+    } else if (uriOrPath.startsWith('/')) {
+      return new Uri.file(uriOrPath, windows: false);
+    }
+
+    // As far as I've seen, Firefox and V8 both always report absolute paths in
+    // their stack frames. However, if we do get a relative path, we should
+    // handle it gracefully.
+    if (uriOrPath.contains('\\')) return path.windows.toUri(uriOrPath);
+    return Uri.parse(uriOrPath);
+  }
+
   Frame(this.uri, this.line, this.column, this.member);
 
   String toString() => '$location in $member';
 }