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

Issue 9372015: Rewrite the negative try-catch tests into a single multi test. (Closed)

Created:
8 years, 10 months ago by kasperl
Modified:
8 years, 10 months ago
Reviewers:
ngeoffray, srdjan
CC:
reviews_dartlang.org, zundel
Visibility:
Public.

Description

Rewrite the negative try-catch tests into a single multi test. R=ngeoffray@google.com,srdjan@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=4069

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -282 lines) Patch
D tests/language/src/TryCatch10NegativeTest.dart View 1 chunk +0 lines, -12 lines 0 comments Download
D tests/language/src/TryCatch1NegativeTest.dart View 1 chunk +0 lines, -54 lines 0 comments Download
D tests/language/src/TryCatch2NegativeTest.dart View 1 chunk +0 lines, -36 lines 0 comments Download
D tests/language/src/TryCatch3NegativeTest.dart View 1 chunk +0 lines, -40 lines 0 comments Download
D tests/language/src/TryCatch4NegativeTest.dart View 1 chunk +0 lines, -40 lines 0 comments Download
D tests/language/src/TryCatch5NegativeTest.dart View 1 chunk +0 lines, -40 lines 0 comments Download
D tests/language/src/TryCatch6NegativeTest.dart View 1 chunk +0 lines, -23 lines 0 comments Download
D tests/language/src/TryCatch7NegativeTest.dart View 1 chunk +0 lines, -24 lines 0 comments Download
D tests/language/src/TryCatch9NegativeTest.dart View 1 chunk +0 lines, -13 lines 0 comments Download
A tests/language/src/TryCatchSyntaxTest.dart View 1 chunk +41 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
kasperl
8 years, 10 months ago (2012-02-09 08:43:36 UTC) #1
ngeoffray
LGTM!
8 years, 10 months ago (2012-02-09 08:48:07 UTC) #2
srdjan
8 years, 10 months ago (2012-02-09 08:58:42 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698