Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2156)

Unified Diff: tests/lib/async/zone_debug_test.dart

Issue 25027004: Add second argument to Future error handlers. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Upload Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tests/lib/async/zone_debug_test.dart
diff --git a/tests/lib/async/zone_debug_test.dart b/tests/lib/async/zone_debug_test.dart
index 10d43fcc2f79f8956f7d0ac759448f5330c7af69..3d41edbef15dbdcfe38e7efce5bd1401114e7c5d 100644
--- a/tests/lib/async/zone_debug_test.dart
+++ b/tests/lib/async/zone_debug_test.dart
@@ -48,7 +48,8 @@ debugZoneRunUnary(Zone self, ZoneDelegate parent, Zone origin, f(arg), arg) {
List expectedDebugTrace;
-debugUncaughtHandler(Zone self, ZoneDelegate parent, Zone origin, error) {
+debugUncaughtHandler(
Lasse Reichstein Nielsen 2013/10/04 09:17:39 void return type?
floitsch 2013/10/05 17:22:59 The return-type is actually specified as being dyn
+ Zone self, ZoneDelegate parent, Zone origin, error, StackTrace stackTrace) {
events.add("handling uncaught error $error");
Expect.listEquals(expectedDebugTrace, restoredStackTrace);
// Suppress the error and don't propagate to parent.

Powered by Google App Engine
This is Rietveld 408576698