Include passing tests in BuildResult.results

tools/gardening/lib/src/compare_failures_impl.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.
 
 /// Compares the test log of a build step with previous builds.
 ///
 /// Use this to detect flakiness of failures, especially timeouts.
 
 import 'dart:async';
 
@@ skipping 14 matching lines @@
     BuildbotClient client, List<String> args,
     {int runCount: 10}) async {
   List<BuildUri> buildUriList = <BuildUri>[];
   for (BuildGroup buildGroup in buildGroups) {
     if (args.contains(buildGroup.groupName)) {
       buildUriList.addAll(buildGroup.createUris(client.mostRecentBuildNumber));
     }
   }
   if (buildUriList.isEmpty) {
     for (String url in args) {
-      if (!url.endsWith('/text')) {
-        // Use the text version of the stdio log.
-        url += '/text';
-      }
-      Uri uri = Uri.parse(url);
-      BuildUri buildUri = new BuildUri(uri);
-      buildUriList.add(buildUri);
+      buildUriList.add(new BuildUri.fromUrl(url));
     }
   }
   Map<BuildUri, List<BuildResult>> buildResults =
       <BuildUri, List<BuildResult>>{};
   for (BuildUri buildUri in buildUriList) {
     List<BuildResult> results =
         await readBuildResults(client, buildUri, runCount);
     buildResults[buildUri] = results;
   }
   return buildResults;
@@ skipping 153 matching lines @@
           sb.write('\n');
         }
         sb.write('\n');
       });
     }
     if (timeoutIds.isEmpty && errorIds.isEmpty) {
       sb.write('No errors found for ${buildUri}');
     }
   }
 }