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

Issue 10981050: Update co19 status (Closed)

Created:
8 years, 2 months ago by ahe
Modified:
8 years, 2 months ago
Reviewers:
ricow1, ngeoffray
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M dart/tests/co19/co19-dart2js.status View 2 chunks +1 line, -1 line 2 comments Download

Messages

Total messages: 4 (0 generated)
ahe
TBR
8 years, 2 months ago (2012-09-27 08:15:14 UTC) #1
ricow1
LGTM
8 years, 2 months ago (2012-09-27 08:29:58 UTC) #2
ngeoffray
LGTM http://codereview.chromium.org/10981050/diff/1/dart/tests/co19/co19-dart2js.status File dart/tests/co19/co19-dart2js.status (right): http://codereview.chromium.org/10981050/diff/1/dart/tests/co19/co19-dart2js.status#newcode113 dart/tests/co19/co19-dart2js.status:113: LibTest/core/List/List_A04_t01: Fail # TODO(ahe): Please triage this failure. ...
8 years, 2 months ago (2012-09-27 08:32:28 UTC) #3
ahe
8 years, 2 months ago (2012-09-27 08:51:14 UTC) #4
http://codereview.chromium.org/10981050/diff/1/dart/tests/co19/co19-dart2js.s...
File dart/tests/co19/co19-dart2js.status (right):

http://codereview.chromium.org/10981050/diff/1/dart/tests/co19/co19-dart2js.s...
dart/tests/co19/co19-dart2js.status:113: LibTest/core/List/List_A04_t01: Fail #
TODO(ahe): Please triage this failure.
On 2012/09/27 08:32:29, ngeoffray wrote:
> It is a co19 issue 224 still. In non-checked mode, it tries to catch an
> IllegalArgumentException, which got renamed to IllegalArgumentError.

Thank you. I think that bug is fixed, and should stop failing next time I update
co19.

Powered by Google App Engine
This is Rietveld 408576698