[Android] Add step text for host_info when failures are detected.

scripts/slave/recipe_modules/chromium_android/api.py

Index: scripts/slave/recipe_modules/chromium_android/api.py
diff --git a/scripts/slave/recipe_modules/chromium_android/api.py b/scripts/slave/recipe_modules/chromium_android/api.py
index 3f875bf7865b03d6cfc2e5570b7b6a14bc00821f..3b668c6ebb8378c9a960885397a9b41912126460 100644
--- a/scripts/slave/recipe_modules/chromium_android/api.py
+++ b/scripts/slave/recipe_modules/chromium_android/api.py
@@ -374,7 +374,7 @@ class AndroidApi(recipe_api.RecipeApi):
     try:
       with self.handle_exit_codes():
         args.extend(['run', '--output', self.m.json.output()])
-        self.m.step(
+        results = self.m.step(
             'Host_Info',
             [self.m.path['checkout'].join('testing', 'scripts',
                                           'host_info.py')] + args,
@@ -382,7 +382,7 @@ class AndroidApi(recipe_api.RecipeApi):
             infra_step=True,
             step_test_data=lambda: self.m.json.test_api.output({
                 'valid': True,
-                'failures': [],
+                'failures': ['Device 3208154b735c5117 blacklisted'],
                 '_host_info': {
                     'os_system': 'os_system',
                     'os_release': 'os_release',
@@ -418,6 +418,12 @@ class AndroidApi(recipe_api.RecipeApi):
                     }]
                 }}),
             **kwargs)
+      if results.json.output.get('failures'):
+        results.presentation.step_text += (

[jbudorick, 2016/07/28 22:43:42]

We should look at doing this in results.presentation.links instead of step_text.

[jbudorick, 2016/07/28 22:46:22]

er, .logs

[rnephew (Reviews Here), 2016/07/28 22:53:28]

Done.

+            self.m.test_utils.format_step_text([
+                ['failures:', results.json.output['failures']]
+        ]))
+      return results
     except self.m.step.InfraFailure:
       pass