Add --step_name option to test.dart.

dart/tools/testing/dart/test_progress.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #library("test_progress");
 
 #import("dart:io");
 #import("test_runner.dart");
 #import("test_suite.dart");
 
 class ProgressIndicator {
   ProgressIndicator(this._startTime, this._printTiming)
       : _tests = [], _failureSummary = [];
 
   factory ProgressIndicator.fromName(String name,
                                      Date startTime,
-                                     bool printTiming) {
+                                     bool printTiming,
+                                     String stepName) {
     switch (name) {
       case 'compact':
         return new CompactProgressIndicator(startTime, printTiming);
       case 'color':
         return new ColorProgressIndicator(startTime, printTiming);
       case 'line':
         return new LineProgressIndicator(startTime, printTiming);
       case 'verbose':
         return new VerboseProgressIndicator(startTime, printTiming);
       case 'silent':
         return new SilentProgressIndicator(startTime, printTiming);
       case 'status':
         return new StatusProgressIndicator(startTime, printTiming);
       case 'buildbot':
-        return new BuildbotProgressIndicator(startTime, printTiming);
+        return new BuildbotProgressIndicator(startTime, printTiming, stepName);
       default:
         assert(false);
         break;
     }
   }
 
   void testAdded() { _foundTests++; }
 
   void start(TestCase test) {
     _printStartProgress(test);
@@ skipping 290 matching lines @@
 
   void _printStartProgress(TestCase test) {
   }
 
   void _printDoneProgress(TestCase test) {
   }
 }
 
 
 class BuildbotProgressIndicator extends ProgressIndicator {
-  BuildbotProgressIndicator(Date startTime, bool printTiming)
+  final String stepName;
+
+  BuildbotProgressIndicator(Date startTime, bool printTiming, this.stepName)
       : super(startTime, printTiming);
 
   void _printStartProgress(TestCase test) {
   }
 
   void _printDoneProgress(TestCase test) {
     var status = 'pass';
     if (test.output.unexpectedOutput) {
       status = 'fail';
     }
     var percent = ((_completedTests() / _foundTests) * 100).toInt().toString();
     print('Done ${test.configurationString} ${test.displayName}: $status');
     print('@@@STEP_CLEAR@@@');
     print('@@@STEP_TEXT@ $percent% +$_passedTests -$_failedTests @@@');
   }
 
   void _printFailureSummary() {
     if (!_failureSummary.isEmpty()) {
-      // We need to create a unique name for each step, otherwise, the
-      // build bot will show the same failure when several steps
-      // fail. We don't know the name of the current step as it is
-      // printed by the program invoking this.
-      // TODO(ahe): Pass in the step name as an option, and let
-      // test.py take care of printing the steps.
-      var name = 'failures ($_startTime)';
       print('@@@STEP_FAILURE@@@');
-      print('@@@BUILD_STEP $name@@@');
+      print('@@@BUILD_STEP $stepName failures@@@');
     }
     super._printFailureSummary();
   }
 }