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

Issue 1508413002: Remove an unnecessary line in a test status file. (Closed)

Created:
5 years ago by Kevin Millikin (Google)
Modified:
5 years ago
Reviewers:
asgerf
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Remove an unnecessary line in a test status file. The syntax of the test status file expects the name of the test which will be used to derive the filename. If the filename is instead given, it matches no test. BUG= R=asgerf@google.com Committed: https://github.com/dart-lang/sdk/commit/deba5aaf7ace0e43f82ec77633d868daa79fda7f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M tests/co19/co19-dart2js.status View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 5 (2 generated)
Kevin Millikin (Google)
5 years ago (2015-12-09 13:44:00 UTC) #2
asgerf
lgtm
5 years ago (2015-12-09 13:44:39 UTC) #3
Kevin Millikin (Google)
5 years ago (2015-12-09 15:28:57 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
deba5aaf7ace0e43f82ec77633d868daa79fda7f (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698