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

Issue 2992363002: Remove three tests that were migrated to language_2 already. (Closed)

Created:
3 years, 4 months ago by Bob Nystrom
Modified:
3 years, 4 months ago
Reviewers:
Jennifer Messerly
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove three tests that were migrated to language_2 already. Committed: https://github.com/dart-lang/sdk/commit/ff563dbe3e946288ad9461b2741782e91979fa18

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -128 lines) Patch
D tests/language/cha_deopt1_test.dart View 1 chunk +0 lines, -48 lines 0 comments Download
D tests/language/cha_deopt2_test.dart View 1 chunk +0 lines, -40 lines 0 comments Download
D tests/language/cha_deopt3_test.dart View 1 chunk +0 lines, -40 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Bob Nystrom
This should help the VM bots.
3 years, 4 months ago (2017-08-05 00:14:55 UTC) #2
Bob Nystrom
Committed patchset #1 (id:1) manually as ff563dbe3e946288ad9461b2741782e91979fa18 (presubmit successful).
3 years, 4 months ago (2017-08-05 00:20:00 UTC) #4
Bob Nystrom
On 2017/08/05 00:14:55, Bob Nystrom wrote: > This should help the VM bots. Landing it ...
3 years, 4 months ago (2017-08-05 00:20:12 UTC) #5
Jennifer Messerly
3 years, 4 months ago (2017-08-05 00:30:20 UTC) #6
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698