Chromium Code Reviews| Index: build/android/pylib/results/json_results.py |
| diff --git a/build/android/pylib/results/json_results.py b/build/android/pylib/results/json_results.py |
| index 4e5c72031a69e4072f857937cebcb94d9811f342..5918caf1317601dee1f82422c33ab794f8a644a5 100644 |
| --- a/build/android/pylib/results/json_results.py |
| +++ b/build/android/pylib/results/json_results.py |
| @@ -87,12 +87,18 @@ def GenerateResultsDict(test_run_results): |
| all_tests = set() |
| per_iteration_data = [] |
| + test_run_links = {} |
| + |
| for test_run_result in test_run_results: |
| iteration_data = collections.defaultdict(list) |
| if isinstance(test_run_result, list): |
| results_iterable = itertools.chain(*(t.GetAll() for t in test_run_result)) |
| + for tr in test_run_result: |
| + test_run_links.update(tr.GetLinks()) |
|
mikecase (-- gone --)
2017/02/01 22:13:45
Removed confusing double dict comprehension from h
|
| + |
| else: |
| results_iterable = test_run_result.GetAll() |
| + test_run_links.update(test_run_result.GetLinks()) |
| for r in results_iterable: |
| result_dict = { |
| @@ -101,8 +107,7 @@ def GenerateResultsDict(test_run_results): |
| 'output_snippet': r.GetLog(), |
| 'losless_snippet': '', |
| 'output_snippet_base64': '', |
| - 'tombstones': r.GetTombstonesUrl() or '', |
| - 'logcat_url': r.GetLogcatUrl() or '', |
| + 'links': r.GetLinks(), |
| } |
| iteration_data[r.GetName()].append(result_dict) |
| @@ -115,6 +120,7 @@ def GenerateResultsDict(test_run_results): |
| # TODO(jbudorick): Add support for disabled tests within base_test_result. |
| 'disabled_tests': [], |
| 'per_iteration_data': per_iteration_data, |
| + 'links': test_run_links, |
| } |