pkg/stack_trace: Use StackTrace.current

test/vm_test.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.
 
 /// This file tests stack_trace's ability to parse live stack traces. It's a
 /// dual of dartium_test.dart, since method names can differ somewhat from
 /// platform to platform. No similar file exists for dart2js since the specific
 /// method names there are implementation details.
 @TestOn('vm')
 
 import 'package:path/path.dart' as path;
 import 'package:stack_trace/stack_trace.dart';
 import 'package:test/test.dart';
 
-String getStackTraceString() {
-  try {
-    throw '';
-  } catch (_, stackTrace) {
-    return stackTrace.toString();
-  }
-}
+String getStackTraceString() => StackTrace.current.toString();
 
-StackTrace getStackTraceObject() {
-  try {
-    throw '';
-  } catch (_, stackTrace) {
-    return stackTrace;
-  }
-}
+StackTrace getStackTraceObject() => StackTrace.current;

[nweiz, 2016/02/01 19:56:29]

Just replace the call sites for these functions.

[kevmoo, 2016/02/01 22:20:07]

Done.

 
 Frame getCaller([int level]) {
   if (level == null) return new Frame.caller();
   return new Frame.caller(level);
 }
 
 Frame nestedGetCaller(int level) => getCaller(level);
 
 Trace getCurrentTrace([int level]) => new Trace.current(level);
 
@@ skipping 72 matching lines @@
 
     test('at level 2 returns the grandparent frame', () {
       expect(nestedGetCaller(2).member, equals('main.<fn>.<fn>'));
     });
 
     test('throws an ArgumentError for negative levels', () {
       expect(() => new Frame.caller(-1), throwsArgumentError);
     });
   });
 }