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

Issue 24008002: Merge tests for dart2js and VM using multi-tests. (Closed)

Created:
7 years, 3 months ago by Michael Lippautz (Google)
Modified:
7 years, 3 months ago
Reviewers:
ahe, rmacnak
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Merge tests for dart2js and VM using multi-tests. BUG= R=rmacnak@google.com Committed: https://code.google.com/p/dart/source/detail?r=27633

Patch Set 1 #

Total comments: 1

Patch Set 2 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -170 lines) Patch
M tests/lib/lib.status View 1 2 chunks +2 lines, -2 lines 0 comments Download
D tests/lib/mirrors/equality_dart2js_test.dart View 1 chunk +0 lines, -112 lines 0 comments Download
M tests/lib/mirrors/equality_test.dart View 3 chunks +14 lines, -5 lines 0 comments Download
D tests/lib/mirrors/parameter_dart2js_test.dart View 1 chunk +0 lines, -50 lines 0 comments Download
M tests/lib/mirrors/parameter_test.dart View 1 2 chunks +10 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Michael Lippautz (Google)
https://codereview.chromium.org/24008002/diff/1/tests/lib/mirrors/equality_test.dart File tests/lib/mirrors/equality_test.dart (right): https://codereview.chromium.org/24008002/diff/1/tests/lib/mirrors/equality_test.dart#newcode98 tests/lib/mirrors/equality_test.dart:98: {'currentMirrorSystem().voidType' : currentMirrorSystem().voidType}, /// 01: ok Please, have mercy ...
7 years, 3 months ago (2013-09-05 22:49:49 UTC) #1
Michael Lippautz (Google)
ping
7 years, 3 months ago (2013-09-17 16:12:51 UTC) #2
rmacnak
lgtm
7 years, 3 months ago (2013-09-18 00:00:36 UTC) #3
Michael Lippautz (Google)
7 years, 3 months ago (2013-09-18 19:50:19 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r27633 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698