Chromium Code Reviews| Index: Tools/Scripts/webkitpy/layout_tests/models/test_run_results.py |
| diff --git a/Tools/Scripts/webkitpy/layout_tests/models/test_run_results.py b/Tools/Scripts/webkitpy/layout_tests/models/test_run_results.py |
| index f203fa534bea2bbdd9c6015dc6172e06caf8116f..a0cac6a19c36cf05e67655472c72259ce7a574d7 100644 |
| --- a/Tools/Scripts/webkitpy/layout_tests/models/test_run_results.py |
| +++ b/Tools/Scripts/webkitpy/layout_tests/models/test_run_results.py |
| @@ -163,7 +163,7 @@ def _interpret_test_failures(failures): |
| return test_dict |
| -def summarize_results(port_obj, expectations, initial_results, retry_results, enabled_pixel_tests_in_retry, only_include_failing=False): |
| +def summarize_results(port_obj, expectations, initial_results, retry_results, enabled_pixel_tests_in_retry, only_include_failing=False, archived_results_path=None, revision_info=''): |
|
Dirk Pranke
2014/06/17 17:58:58
it feels a bit odd to me that the results object w
patro
2014/07/15 10:36:57
Yes, things can be seperate there need not be any
|
| """Returns a dictionary containing a summary of the test runs, with the following fields: |
| 'version': a version indicator |
| 'fixable': The number of fixable tests (NOW - PASS) |
| @@ -306,6 +306,12 @@ def summarize_results(port_obj, expectations, initial_results, retry_results, en |
| results['build_number'] = port_obj.get_option('build_number') |
| results['builder_name'] = port_obj.get_option('builder_name') |
| + if port_obj.get_option("enable_versioned_results") and archived_results_path: |
| + results['previous_results_location'] = archived_results_path |
| + results['chromium_revision'] = '' |
| + results['blink_revision'] = '' |
| + |
| + |
| try: |
| # Don't do this by default since it takes >100ms. |
| # It's only used for uploading data to the flakiness dashboard. |