Add --step_name option to test.dart.

dart/tools/test.dart

Index: dart/tools/test.dart
diff --git a/dart/tools/test.dart b/dart/tools/test.dart
index e73685acf32949c54d31ff8686199c40c95b919c..d4eb44cd53d864e498377f859b8db2bf4f077c49 100755
--- a/dart/tools/test.dart
+++ b/dart/tools/test.dart
@@ -75,10 +75,15 @@ main() {
   Map<String, RegExp> selectors = firstConf['selectors'];
   var maxProcesses = firstConf['tasks'];
   var progressIndicator = firstConf['progress'];
+  var stepName = firstConf['step_name'];

[Bill Hesse, 2012/09/11 10:26:53]

How about BuildbotProgressIndicator.stepName = firstConf['step_name'].  Or a
setter.  Then it doesn't need to be passed as a parameter through two unrelated
class constructors.

   var verbose = firstConf['verbose'];
   var printTiming = firstConf['time'];
   var listTests = firstConf['list'];
 
+  if (progressIndicator == 'buildbot' && stepName !== null) {

[ricow1, 2012/09/11 08:14:38]

I am not particularly happy about having buildbot related stuff in the main
general purpose test script - even if it is only a print statement.

+    print('@@@BUILD_STEP $stepName@@@');
+  }
+
   // Print the configurations being run by this execution of
   // test.dart. However, don't do it if the silent progress indicator
   // is used. This is only needed because of the junit tests.
@@ -132,5 +137,6 @@ main() {
                                printTiming,
                                enqueueConfiguration,
                                verbose,
-                               listTests);
+                               listTests,
+                               stepName);
 }