Include build revision in BuildResult

tools/gardening/lib/src/buildbot_loading.dart

 // Copyright (c) 2017, 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.
 
 import 'dart:async';
 import 'dart:io';
 import 'util.dart';
 
 import 'buildbot_structures.dart';
 import 'cache.dart';
 import 'logdog.dart';
 
 const String BUILDBOT_BUILDNUMBER = ' BUILDBOT_BUILDNUMBER: ';
+const String BUILDBOT_REVISION = ' BUILDBOT_REVISION: ';
 
 /// Read the build result for [buildUri].
 ///
 /// The data is loaded from the cache, if available, otherwise [read] is called
 /// to fetch the data and stored in the cache afterwards.
 Future<BuildResult> _readBuildResult(
     BuildUri buildUri, Future<String> read()) async {
   if (buildUri.buildNumber < 0) {
     String text = await read();
     BuildResult result = parseTestStepResult(buildUri, text);
@@ skipping 47 matching lines @@
         new TestConfiguration(configName, archName, testName), status);
   } catch (_) {
     return null;
   }
 }
 
 /// Parses the [buildUri] test log and creates a [BuildResult] for it.
 BuildResult parseTestStepResult(BuildUri buildUri, String text) {
   log('Parsing results: $buildUri (${text.length} bytes)');
   int buildNumber;
+  String buildRevision;
   List<String> currentFailure;
   bool parsingTimingBlock = false;
 
   List<TestStatus> results = <TestStatus>[];
   List<TestFailure> failures = <TestFailure>[];
   List<Timing> timings = <Timing>[];
   for (String line in text.split('\n')) {
     if (line.startsWith(BUILDBOT_BUILDNUMBER)) {
       buildNumber =
           int.parse(line.substring(BUILDBOT_BUILDNUMBER.length).trim());
       buildUri = buildUri.withBuildNumber(buildNumber);
     }
+    if (line.startsWith(BUILDBOT_REVISION)) {
+      buildRevision = line.substring(BUILDBOT_REVISION.length).trim();
+    }
     if (currentFailure != null) {
       if (line.startsWith('Done ')) {
         TestStatus status = parseTestStatus(line);
         if (status != null) {
           results.add(status);
           failures.add(new TestFailure(buildUri, currentFailure));
           currentFailure = null;
         }
       } else {
         currentFailure.add(line);
@@ skipping 11 matching lines @@
       parsingTimingBlock = true;
     } else if (parsingTimingBlock) {
       if (line.startsWith('0:')) {
         timings.addAll(parseTimings(buildUri, line));
       } else {
         parsingTimingBlock = false;
       }
     }
   }
   return new BuildResult(buildUri, buildNumber ?? buildUri.absoluteBuildNumber,
-      results, failures, timings);
+      buildRevision, results, failures, timings);
 }
 
 /// Create the [Timing]s for the [line] as found in the top-20 timings of a
 /// build step stdio log.
 List<Timing> parseTimings(BuildUri uri, String line) {
   List<String> parts = split(line, [' - ', ' - ', ' ']);
   String time = parts[0];
   String stepName = parts[1];
   String configName = parts[2];
   String testNames = parts[3];
   List<Timing> timings = <Timing>[];
   for (String name in testNames.split(',')) {
     name = name.trim();
     int slashPos = name.indexOf('/');
     String archName = name.substring(0, slashPos);
     String testName = name.substring(slashPos + 1);
     timings.add(new Timing(
         uri,
         time,
         new TestStep(
             stepName, new TestConfiguration(configName, archName, testName))));
   }
   return timings;
 }