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

Issue 9034022: Include compiler tests suite, tweaks for tests. (Closed)

Created:
8 years, 12 months ago by scheglov
Modified:
8 years, 12 months ago
Reviewers:
codefu
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Include compiler tests suite, tweaks for tests. R=codefu@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=2877

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -14 lines) Patch
A compiler/javatests/com/google/dart/compiler/CompilerTests.java View 1 chunk +23 lines, -0 lines 0 comments Download
M compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java View 8 chunks +19 lines, -9 lines 0 comments Download
M compiler/javatests/com/google/dart/compiler/SystemLibraryManagerTest.java View 6 chunks +18 lines, -5 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
scheglov
8 years, 12 months ago (2011-12-29 17:40:13 UTC) #1
codefu
8 years, 12 months ago (2011-12-29 20:17:16 UTC) #2
On 2011/12/29 17:40:13, scheglov wrote:

LGTM

Powered by Google App Engine
This is Rietveld 408576698