[Findit] Adding support for extracting revisions from dict instead of list

appengine/findit/common/buildbucket_client.py

 # Copyright 2015 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.
 
 """"Serves as a client for selected APIs in Buildbucket."""
 
 import collections
 import json
 import logging
 
@@ skipping 64 matching lines @@
   SCHEDULED = 'SCHEDULED'
   STARTED = 'STARTED'
   COMPLETED = 'COMPLETED'
 
   def __init__(self, raw_json_data):
     self.id = raw_json_data.get('id')
     self.url = raw_json_data.get('url')
     self.status = raw_json_data.get('status')
     result_details_json = json.loads(
         raw_json_data.get('result_details_json', '{}')) or {}
-    self.result = result_details_json.get('properties', {}).get('result', [])
+    self.report = (result_details_json.get('properties', {}).get('result') or
+                   result_details_json.get('properties', {}).get('report', {}))
 
 
 class BuildbucketError(object):
   """Represents an error returned by Buildbucket."""
 
   # Error reasons.
   BUILD_NOT_FOUND = 'BUILD_NOT_FOUND'
   INVALID_INPUT = 'INVALID_INPUT'
 
   def __init__(self, raw_json_data):
@@ skipping 42 matching lines @@
   }
 
   for try_job in try_jobs:
     status_code, content = HttpClient().Put(
         _BUILDBUCKET_PUT_GET_ENDPOINT,
         json.dumps(try_job.ToBuildbucketRequest()), headers=headers)
     if status_code == 200:  # pragma: no cover
       json_results.append(json.loads(content))
     else:
       error_content = {
-        'error': {
-          'reason': status_code,
-          'message': content
-        }
+          'error': {
+              'reason': status_code,
+              'message': content
+          }
       }
       json_results.append(error_content)
 
   return _ConvertFuturesToResults(json_results)
 
 
 def GetTryJobs(build_ids):
   """Returns the try-job builds for the given build ids.
 
   Args:
@@ skipping 10 matching lines @@
       'Content-Type': 'application/json; charset=UTF-8'
   }
 
   for build_id in build_ids:
     status_code, content = HttpClient().Get(
         _BUILDBUCKET_PUT_GET_ENDPOINT + '/' + build_id, headers=headers)
     if status_code == 200:  # pragma: no cover
       json_results.append(json.loads(content))
     else:
       error_content = {
-        'error': {
-          'reason': status_code,
-          'message': content
-        }
+          'error': {
+              'reason': status_code,
+              'message': content
+          }
       }
       json_results.append(error_content)
 
   return _ConvertFuturesToResults(json_results)