[Findit] Refactoring compile recipe return results into report dict instead of list

scripts/slave/recipes/findit/chromium/compile.py

Index: scripts/slave/recipes/findit/chromium/compile.py
diff --git a/scripts/slave/recipes/findit/chromium/compile.py b/scripts/slave/recipes/findit/chromium/compile.py
index ef3a929ca8c3fb03ba89949f96645f978e2b3738..07dd0f75cb7950ca36c5610a4e8e2c62cd199979 100644
--- a/scripts/slave/recipes/findit/chromium/compile.py
+++ b/scripts/slave/recipes/findit/chromium/compile.py
@@ -10,15 +10,15 @@ from recipe_engine.recipe_api import Property
 
 
 DEPS = [
-  'chromium',
-  'chromium_tests',
-  'findit',
-  'gclient',
-  'recipe_engine/json',
-  'recipe_engine/path',
-  'recipe_engine/properties',
-  'recipe_engine/python',
-  'recipe_engine/step',
+    'chromium',
+    'chromium_tests',
+    'findit',
+    'gclient',
+    'recipe_engine/json',
+    'recipe_engine/path',
+    'recipe_engine/properties',
+    'recipe_engine/python',
+    'recipe_engine/step',
 ]
 
 
@@ -52,10 +52,8 @@ def _run_compile_at_revision(api, target_mastername, target_buildername,
     # Checkout code at the given revision to recompile.
     bot_config = api.chromium_tests.create_bot_config_object(
         target_mastername, target_buildername)
-    bot_update_step, bot_db = \
-        api.chromium_tests.prepare_checkout(
-            bot_config,
-            root_solution_revision=revision)
+    bot_update_step, bot_db = api.chromium_tests.prepare_checkout(
+        bot_config, root_solution_revision=revision)
 
     # TODO(http://crbug.com/560991): if compile targets are provided, check
     # whether they exist and then use analyze to compile the impacted ones by
@@ -125,7 +123,7 @@ def RunSteps(api, target_mastername, target_buildername,
       if compile_result == CompileResult.FAILED:
         # TODO(http://crbug.com/560991): if compile targets are specified,
         # compile may fail because those targets are added in a later revision.
-        break # Found the culprit, no need to check later revisions.
+        break  # Found the culprit, no need to check later revisions.
   finally:
     # Report the result.
     # TODO(http://crbug.com/563807): use api.python.succeeding_step instead.
@@ -140,6 +138,8 @@ def RunSteps(api, target_mastername, target_buildername,
     # Set the result as a build property too, so that it will be reported back
     # to Buildbucket and Findit will pull from there instead of buildbot master.
     step_result.presentation.properties['result'] = results
+    step_result.presentation.properties['number_of_commits_in_range'] = len(

[stgao, 2016/01/25 22:21:35]

As discussed offline, we'd better not add a new build properties.
And instead, we could change the ``results`` into a dict and add more
information into it.

[lijeffrey, 2016/01/29 09:53:42]

Done.

+        revisions_to_check)

[chanli, 2016/01/22 18:05:27]

I think this number is actually
len(failure_info['builds'][current_failure]['blame_list']), right?

[lijeffrey, 2016/01/22 22:26:43]

I think you're right, it should be the same. In the case we use failure_info,
then this CL is not needed. If we use that approach then we can re factor
ScheduleTryJobPipeline and MonitorTryJobPipeline to consume failure_info instead
of master_name, builder_name, etc which it can get from failure_info (I plan to
use MonitorTryJobPipeline to handle updating WfTryJobData). The downside is if
for whatever reason if there's a discrepancy between what the try job derived as
the list of revisions to check and what's in failure_info's blame list then the
data stored may not be representative of what the try job would have tried in a
worst-case scenario. What do you guys think?

 
   return results
 
@@ -171,11 +171,11 @@ def GenTests(api):
       props() +
       api.override_step_data('test r1.read test spec',
                              api.json.output({
-                                'Linux Builder': {
-                                   'additional_compile_targets': [
-                                       'base_unittests',
-                                   ],
-                                }
+                                 'Linux Builder': {
+                                     'additional_compile_targets': [
+                                         'base_unittests',
+                                     ],
+                                 }
                              }))
   )
 
@@ -226,12 +226,12 @@ def GenTests(api):
       props(use_analyze=True) +
       api.override_step_data('test r1.read test spec',
                              api.json.output({
-                                'Linux Builder': {
-                                   'additional_compile_targets': [
-                                       'a', 'a_run',
-                                       'b', 'b_run',
-                                   ],
-                                }
+                                 'Linux Builder': {
+                                     'additional_compile_targets': [
+                                         'a', 'a_run',
+                                         'b', 'b_run',
+                                     ],
+                                 }
                              })) +
       api.override_step_data(
           'test r1.analyze',