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

Issue 1275003002: dart2js: Add a failing test case for the async rewriter. (Closed)

Created:
5 years, 4 months ago by karlklose
Modified:
5 years, 4 months ago
Reviewers:
sigurdm
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

dart2js: Add a failing test case for the async rewriter. R=sigurdm@google.com Committed: https://github.com/dart-lang/sdk/commit/4bd727ff6c4be5419380f07cddbfa829c36886dc

Patch Set 1 #

Total comments: 1

Patch Set 2 : Address comment. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+48 lines, -0 lines) Patch
M tests/compiler/dart2js/async_await_js_transform_test.dart View 1 chunk +46 lines, -0 lines 0 comments Download
M tests/compiler/dart2js/dart2js.status View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
karlklose
5 years, 4 months ago (2015-08-06 10:37:36 UTC) #1
sigurdm
LGTM https://codereview.chromium.org/1275003002/diff/1/tests/compiler/dart2js/dart2js.status File tests/compiler/dart2js/dart2js.status (right): https://codereview.chromium.org/1275003002/diff/1/tests/compiler/dart2js/dart2js.status#newcode7 tests/compiler/dart2js/dart2js.status:7: async_await_js_transform_test/01: Fail # Issue 23997 Could be RuntimeError.
5 years, 4 months ago (2015-08-06 10:50:43 UTC) #2
karlklose
5 years, 4 months ago (2015-08-06 10:55:24 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
4bd727ff6c4be5419380f07cddbfa829c36886dc (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698