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

Issue 10905206: Update co19 to r336. (Closed)

Created:
8 years, 3 months ago by ahe
Modified:
8 years, 3 months ago
Reviewers:
kasperl
CC:
reviews_dartlang.org, Ivan Posva
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : Rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+181 lines, -154 lines) Patch
M dart/tests/co19/co19-compiler.status View 5 chunks +28 lines, -33 lines 0 comments Download
M dart/tests/co19/co19-dart2js.status View 8 chunks +73 lines, -42 lines 0 comments Download
M dart/tests/co19/co19-runtime.status View 6 chunks +79 lines, -78 lines 0 comments Download
M deps/all.deps/DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
ahe
8 years, 3 months ago (2012-09-11 06:31:50 UTC) #1
kasperl
LGTM. https://chromiumcodereview.appspot.com/10905206/diff/1/dart/tests/co19/co19-compiler.status File dart/tests/co19/co19-compiler.status (right): https://chromiumcodereview.appspot.com/10905206/diff/1/dart/tests/co19/co19-compiler.status#newcode778 dart/tests/co19/co19-compiler.status:778: Language/07_Classes/2_Getters_A02_t01: Fail, Pass Do you know why a ...
8 years, 3 months ago (2012-09-11 06:34:48 UTC) #2
ahe
I have filed http://dartbug.com/5064 as the analyzer is effectively ignoring more than 1000 tests. https://chromiumcodereview.appspot.com/10905206/diff/1/dart/tests/co19/co19-compiler.status ...
8 years, 3 months ago (2012-09-11 07:09:25 UTC) #3
scheglov
8 years, 3 months ago (2012-09-11 13:04:26 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698