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

Issue 12114027: If we made progress, add testCases to the front of the queue in test_runner.dart (Closed)

Created:
7 years, 10 months ago by kustermann
Modified:
7 years, 10 months ago
Reviewers:
ricow1, ahe, ngeoffray
CC:
reviews_dartlang.org
Visibility:
Public.

Description

If we made progress, add testCases to the front of the queue in test_runner.dart Committed: https://code.google.com/p/dart/source/detail?r=17913

Patch Set 1 #

Patch Set 2 : #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M tools/testing/dart/test_runner.dart View 1 1 chunk +1 line, -1 line 2 comments Download

Messages

Total messages: 5 (0 generated)
kustermann
It's was actually way easier than I thought.
7 years, 10 months ago (2013-01-31 12:52:24 UTC) #1
ngeoffray
https://codereview.chromium.org/12114027/diff/2001/tools/testing/dart/test_runner.dart File tools/testing/dart/test_runner.dart (right): https://codereview.chromium.org/12114027/diff/2001/tools/testing/dart/test_runner.dart#newcode1668 tools/testing/dart/test_runner.dart:1668: _tests.addFirst(testCase); I thought you could not do this because ...
7 years, 10 months ago (2013-01-31 12:56:31 UTC) #2
ricow1
LGTM
7 years, 10 months ago (2013-01-31 12:58:51 UTC) #3
kustermann
https://codereview.chromium.org/12114027/diff/2001/tools/testing/dart/test_runner.dart File tools/testing/dart/test_runner.dart (right): https://codereview.chromium.org/12114027/diff/2001/tools/testing/dart/test_runner.dart#newcode1668 tools/testing/dart/test_runner.dart:1668: _tests.addFirst(testCase); On 2013/01/31 12:56:31, ngeoffray wrote: > I thought ...
7 years, 10 months ago (2013-01-31 13:01:17 UTC) #4
ngeoffray
7 years, 10 months ago (2013-01-31 13:02:45 UTC) #5
LGTM :)

Powered by Google App Engine
This is Rietveld 408576698