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

test/source_map_stack_trace_test.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:source_span/source_span.dart';
 import 'package:stack_trace/stack_trace.dart';
 import 'package:source_map_stack_trace/source_map_stack_trace.dart';
 import 'package:test/test.dart';
 
 /// A simple [Mapping] for tests that don't need anything special.
 final _simpleMapping = parseJson(
     (new SourceMapBuilder()
         ..addSpan(
@@ skipping 39 matching lines @@
 foo.dart.js 10:11  baz
 """);
 
     var frames = _mapTrace(_simpleMapping, trace).frames;
 
     expect(frames.length, equals(2));
     expect(frames.first.member, equals("foo"));
     expect(frames.last.member, equals("baz"));
   });
 
+  test("include JS frames from external libraries with source map bundle", () {
+    var trace = new Trace.parse("""
+foo.dart.js 10:11  foo
+jquery.js 10:1  foo
+""");
+    var builder = new SourceMapBuilder()
+      ..addSpan(
+          new SourceMapSpan.identifier(
+              new SourceLocation(1,
+                  line: 1, column: 3, sourceUrl: "packages/foo/foo.dart"),
+              "qux"),
+          new SourceSpan(new SourceLocation(8, line: 5, column: 0),
+              new SourceLocation(12, line: 9, column: 1), "\n" * 4));
+    var sourceMapJson = builder.build("foo.dart.js.map");
+    sourceMapJson['file'] = "foo.dart.js";
+
+    var bundle = [sourceMapJson];
+    var mapping = parseJsonExtended(bundle);
+    var frames = _mapTrace(mapping, trace,
+            packageResolver: new SyncPackageResolver.root("packages/"))
+        .frames;
+
+    expect(frames.length, equals(2));
+
+    var frame = frames.first;
+    expect(frame.uri, equals(Uri.parse("package:foo/foo.dart")));
+    expect(frame.line, equals(2));
+    expect(frame.column, equals(4));
+
+    frame = frames.last;
+    expect(p.basename(frame.uri.toString()), equals("jquery.js"));
+    expect(frame.line, equals(10));
+  });
+
   test("falls back to column 0 for unlisted column", () {
     var trace = new Trace.parse("foo.dart.js 10  foo");
     var builder = new SourceMapBuilder()
         ..addSpan(
             new SourceMapSpan.identifier(
                 new SourceLocation(1,
                     line: 1, column: 3, sourceUrl: "foo.dart"),
                 "qux"),
             new SourceSpan(
                 new SourceLocation(8, line: 5, column: 0),
@@ skipping 186 matching lines @@
   return new Chain.forTrace(mapStackTrace(sourceMap, stackTrace,
       minified: minified, packageResolver: packageResolver, sdkRoot: sdkRoot,
       packageRoot: packageRoot));
 }
 
 /// Runs the mapper's prettification logic on [member] and returns the result.
 String _prettify(String member) {
   var trace = new Trace([new Frame(Uri.parse("foo.dart.js"), 10, 11, member)]);
   return _mapTrace(_simpleMapping, trace).frames.first.member;
 }