Fix behavior causing frames lacking a source map to always be omitted. Add `includeUnmappedFrames` …

lib/source_map_stack_trace.dart

 // Copyright (c) 2015, 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.
 
 import 'package:package_resolver/package_resolver.dart';
 import 'package:path/path.dart' as p;
 import 'package:source_maps/source_maps.dart';
 import 'package:stack_trace/stack_trace.dart';
 
 /// Convert [stackTrace], a stack trace generated by dart2js-compiled
@@ skipping 44 matching lines @@
   return new Trace(trace.frames.map((frame) {
     // If there's no line information, there's no way to translate this frame.
     // We could return it as-is, but these lines are usually not useful anyways.
     if (frame.line == null) return null;
 
     // If there's no column, try using the first column of the line.
     var column = frame.column == null ? 0 : frame.column;
 
     // Subtract 1 because stack traces use 1-indexed lines and columns and
     // source maps uses 0-indexed.
-    var span = sourceMap.spanFor(frame.line - 1, column - 1);
+    var span =
+        sourceMap.spanFor(frame.line - 1, column - 1, uri: frame.uri?.toString());

[nweiz, 2016/12/12 22:04:56]

Long line.

You're using a new feature in the source_maps package, which means you need to
bump the dependency on source_maps to ensure that we only ever get versions that
have the new feature.

[Jacob, 2016/12/13 18:01:53]

Done.

 
     // If we can't find a source span, ignore the frame. It's probably something
     // internal that the user doesn't care about.
     if (span == null) return null;
 
     var sourceUrl = span.sourceUrl.toString();
     if (sdkRoot != null && p.url.isWithin(sdkLib, sourceUrl)) {
       sourceUrl = "dart:" + p.url.relative(sourceUrl, from: sdkLib);
     } else if (packageResolver != null) {
       if (packageResolver.packageRoot != null &&
@@ skipping 45 matching lines @@
       .replaceAll(new RegExp(r"[a-zA-Z_0-9]+\$"), "")
       // Get rid of the static method prefix. The class name also exists in the
       // invocation, so we're not getting rid of any information.
       .replaceAll(new RegExp(r"^[a-zA-Z_0-9]+.(static|dart)."), "")
       // Convert underscores after identifiers to dots. This runs the risk of
       // incorrectly converting members that contain underscores, but those are
       // contrary to the style guide anyway.
       .replaceAllMapped(new RegExp(r"([a-zA-Z0-9]+)_"),
           (match) => match[1] + ".");
 }