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

Issue 1136383004: Add test for Issue 23432 (Closed)

Created:
5 years, 7 months ago by sra1
Modified:
5 years, 6 months ago
CC:
reviews_dartlang.org
Target Ref:
refs/remotes/git-svn
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : address code review comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M tests/compiler/dart2js_extra/23432_test.dart View 1 2 chunks +5 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (1 generated)
sra1
5 years, 7 months ago (2015-05-14 20:04:32 UTC) #2
Siggi Cherem (dart-lang)
lgtm https://codereview.chromium.org/1136383004/diff/1/tests/compiler/dart2js_extra/23432_test.dart File tests/compiler/dart2js_extra/23432_test.dart (right): https://codereview.chromium.org/1136383004/diff/1/tests/compiler/dart2js_extra/23432_test.dart#newcode5 tests/compiler/dart2js_extra/23432_test.dart:5: // Regression test for http://dartbug.com/23432 let's add a ...
5 years, 7 months ago (2015-05-14 23:23:05 UTC) #3
sra1
Committed patchset #1 (id:1) manually as 45821 (presubmit successful).
5 years, 7 months ago (2015-05-15 19:59:37 UTC) #4
sra1
5 years, 7 months ago (2015-05-15 20:31:03 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as 45824 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698