Make the StackTrace library better handle core library frames.

pkg/stack_trace/lib/src/trace.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 trace;
 
 import 'dart:collection';
 import 'dart:math' as math;
 
 import 'frame.dart';
 import 'lazy_trace.dart';
 
-final _patchRegExp = new RegExp(r"-patch$");
+final _terseRegExp = new RegExp(r"(-patch)?(/.*)?$");
 
 /// A stack trace, comprised of a list of stack frames.
 class Trace implements StackTrace {
   /// The stack frames that comprise this stack trace.
   final List<Frame> frames;
 
   /// Returns a human-readable representation of [stackTrace]. If [terse] is
   /// set, this folds together multiple stack frames from the Dart core
   /// libraries, so that only the core library method directly called from user
   /// code is visible (see [Trace.terse]).
@@ skipping 48 matching lines @@
   /// This is identical to [toString]. It will not be formatted in the manner of
   /// native stack traces.
   String get stackTrace => toString();
 
   /// Returns a string representation of this stack trace.
   ///
   /// This is identical to [toString]. It will not be formatted in the manner of
   /// native stack traces.
   String get fullStackTrace => toString();
 
-  /// Returns a terser version of [this]. This is accomplished by folding
-  /// together multiple stack frames from the core library, as in [foldFrames].
-  /// Core library patches are also renamed to remove their `-patch` suffix.
+  /// Returns a terser version of [this].
+  ///
+  /// This is accomplished by folding together multiple stack frames from the
+  /// core library, as in [foldFrames]. Remaining core library frames have their
+  /// libraries, "-patch" suffixes, and line numbers removed.
   Trace get terse {
     return new Trace(foldFrames((frame) => frame.isCore).frames.map((frame) {
       if (!frame.isCore) return frame;
-      var library = frame.library.replaceAll(_patchRegExp, '');
-      return new Frame(
-          Uri.parse(library), frame.line, frame.column, frame.member);
+      var library = frame.library.replaceAll(_terseRegExp, '');
+      return new Frame(Uri.parse(library), null, null, frame.member);
     }));
   }
 
   /// Returns a new [Trace] based on [this] where multiple stack frames matching
   /// [predicate] are folded together. This means that whenever there are
   /// multiple frames in a row that match [predicate], only the last one is
   /// kept.
   ///
   /// This is useful for limiting the amount of library code that appears in a
   /// stack trace by only showing user code and code that's called by user code.
@@ skipping 31 matching lines @@
   if (string.length >= length) return string;
 
   var result = new StringBuffer();
   result.write(string);
   for (var i = 0; i < length - string.length; i++) {
     result.write(' ');
   }
 
   return result.toString();
 }