rebaseline-cl: Enable filling in missing results if no results could be fetched.

third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py

Index: third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
index eeef1f691152de803998a1d6edf5b41a905a7afd..dee73383de83a544d6814d417b588d6846ce2969 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
@@ -77,8 +77,6 @@ class RebaselineCL(AbstractParallelRebaselineCommand):
             return 1
 
         if builders_with_no_results and not options.fill_missing:
-            # TODO(qyearsley): Support trying to continue as long as there are
-            # some results from some builder; see http://crbug.com/673966.
             _log.error('The following builders have no results:')
             for builder in builders_with_no_results:
                 _log.error('  %s', builder)
@@ -86,13 +84,15 @@ class RebaselineCL(AbstractParallelRebaselineCommand):
 
         _log.debug('Getting results for issue %d.', issue_number)
         builds_to_results = self._fetch_results(builds)
-        if builds_to_results is None:
+        if not options.fill_missing and len(builds_to_results) < len(builds):
             return 1
 
         test_baseline_set = TestBaselineSet(tool)
         if args:
             for test in args:
                 for build in builds:
+                    if not builds_to_results.get(build):
+                        continue
                     test_baseline_set.add(test, build)
         else:
             test_baseline_set = self._make_test_baseline_set(
@@ -160,10 +160,10 @@ class RebaselineCL(AbstractParallelRebaselineCommand):
             results_url = buildbot.results_url(build.builder_name, build.build_number)
             layout_test_results = buildbot.fetch_results(build)
             if layout_test_results is None:
-                _log.error('Failed to fetch results for: %s', build)
-                _log.error('Results were expected to exist at:\n%s/results.html', results_url)
-                _log.error('If the job failed, you could retry by running:\ngit cl try -b %s', build.builder_name)
-                return None
+                _log.info('Failed to fetch results for %s', build)
+                _log.info('Results URL: %s/results.html', results_url)
+                _log.info('Retry job by running: git cl try -b %s', build.builder_name)
+                continue
             results[build] = layout_test_results
         return results
 
@@ -238,6 +238,7 @@ class RebaselineCL(AbstractParallelRebaselineCommand):
         a Win10 builder which does have results.
         """
         all_ports = {self._tool.builders.port_name_for_builder_name(b) for b in self._try_bots()}
+        print test_baseline_set

[wkorman, 2017/04/13 16:45:53]

debugging?

         for test_prefix in test_baseline_set.test_prefixes():
             build_port_pairs = test_baseline_set.build_port_pairs(test_prefix)
             missing_ports = all_ports - {p for _, p in build_port_pairs}