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

Issue 10972007: Update co19 to r343. (Closed)

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

Description

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+103 lines, -1087 lines) Patch
M dart/tests/co19/co19-compiler.status View 13 chunks +41 lines, -263 lines 0 comments Download
M dart/tests/co19/co19-dart2js.status View 19 chunks +24 lines, -410 lines 0 comments Download
M dart/tests/co19/co19-runtime.status View 16 chunks +35 lines, -412 lines 0 comments Download
M dart/tools/testing/dart/scrub_status_file.dart View 1 chunk +2 lines, -1 line 2 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, 2 months ago (2012-09-24 11:46:22 UTC) #1
ngeoffray
LGTM
8 years, 2 months ago (2012-09-24 11:56:07 UTC) #2
Lasse Reichstein Nielsen
lgtm https://codereview.chromium.org/10972007/diff/1/dart/tools/testing/dart/scrub_status_file.dart File dart/tools/testing/dart/scrub_status_file.dart (right): https://codereview.chromium.org/10972007/diff/1/dart/tools/testing/dart/scrub_status_file.dart#newcode20 dart/tools/testing/dart/scrub_status_file.dart:20: // TODO(ahe): Enable this when dart2dart stops using ...
8 years, 2 months ago (2012-09-24 11:56:32 UTC) #3
ahe
8 years, 2 months ago (2012-09-24 12:53:39 UTC) #4
Thank you!

https://codereview.chromium.org/10972007/diff/1/dart/tools/testing/dart/scrub...
File dart/tools/testing/dart/scrub_status_file.dart (right):

https://codereview.chromium.org/10972007/diff/1/dart/tools/testing/dart/scrub...
dart/tools/testing/dart/scrub_status_file.dart:20: // TODO(ahe): Enable this
when dart2dart stops using *.
On 2012/09/24 11:56:32, Lasse Reichstein Nielsen wrote:
> Stops using multiplication?

See http://dartbug.com/5401

Powered by Google App Engine
This is Rietveld 408576698