Cause chromium_android recipe to consume device_status_check's json.

scripts/slave/recipe_modules/chromium_android/api.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import os
 import urllib
 
 from slave import recipe_api
 
 class AndroidApi(recipe_api.RecipeApi):
   def __init__(self, **kwargs):
     super(AndroidApi, self).__init__(**kwargs)
+    self._devices = None
 
   def get_config_defaults(self):
     return {
       'REVISION': self.m.properties.get('revision', '')
     }
 
   def get_env(self):
     return self.m.chromium.get_env()
 
   @property
+  def devices(self):
+    assert self._devices is not None,\
+        'devices is only available after device_status_check()'
+    return self._devices
+
+  @property
   def out_path(self):
     return self.m.path['checkout'].join('out')
 
   @property
   def coverage_dir(self):
     return self.out_path.join(self.c.BUILD_CONFIG, 'coverage')
 
   def configure_from_properties(self, config_name, **kwargs):
     self.set_config(config_name, **kwargs)
 
@@ skipping 204 matching lines @@
         env=self.m.chromium.get_env(),
         infra_step=True)
 
   def detect_and_setup_devices(self, restart_usb=False, skip_wipe=False,
                                disable_location=False):
     self.device_status_check(restart_usb=restart_usb)
     self.provision_devices(
       skip_wipe=skip_wipe, disable_location=disable_location)
 
   def device_status_check(self, restart_usb=False, **kwargs):
-    args = []
+    args = ['--json-output', self.m.json.output()]
     if restart_usb:
-      args = ['--restart-usb']
+      args += ['--restart-usb']
 
     try:
-      self.m.step(
+      result = self.m.step(
           'device_status_check',
           [self.m.path['checkout'].join('build', 'android', 'buildbot',
                                 'bb_device_status_check.py')] + args,
+          step_test_data=lambda: self.m.json.test_api.output([{
+              "battery": {
+                  "status": "5",
+                  "scale": "100",
+                  "temperature": "249",
+                  "level": "100",
+                  "AC powered": "false",
+                  "health": "2",
+                  "voltage": "4286",
+                  "Wireless powered": "false",
+                  "USB powered": "true",
+                  "technology": "Li-ion",
+                  "present": "true"
+              },
+              "wifi_ip": "",
+              "imei_slice": "Unknown",
+              "build": "LRX21O",
+              "build_detail":
+                  "google/razor/flo:5.0/LRX21O/1570415:userdebug/dev-keys",
+              "serial": "07a00ca4",
+              "type": "flo"
+          }]),
           env=self.m.chromium.get_env(),
           infra_step=True,
           **kwargs)
+      self._devices = [d['serial'] for d in result.json.output]
     except self.m.step.InfraFailure as f:
       params = {
         'summary': ('Device Offline on %s %s' %
           (self.m.properties['mastername'], self.m.properties['slavename'])),
         'comment': ('Buildbot: %s\n(Please do not change any labels)' %
           self.m.properties['buildername']),
         'labels': 'Restrict-View-Google,OS-Android,Infra,Infra-Labs',
       }
       link = ('https://code.google.com/p/chromium/issues/entry?%s' %
         urllib.urlencode(params))
@@ skipping 328 matching lines @@
         **kwargs)
 
     self.m.gsutil.upload(
         source=self.coverage_dir.join('coverage_html'),
         bucket='chrome-code-coverage',
         dest=gs_dest,
         args=['-R'],
         name='upload coverage report',
         link_name='Coverage report',
         **kwargs)