Index: tools/telemetry/telemetry/results/page_test_results.py |
diff --git a/tools/telemetry/telemetry/results/page_test_results.py b/tools/telemetry/telemetry/results/page_test_results.py |
index 8d960a2382e7c2134fc33e969666a2df869409e3..6b6c09acca861d817819be1b52ea2d89a03a90f5 100644 |
--- a/tools/telemetry/telemetry/results/page_test_results.py |
+++ b/tools/telemetry/telemetry/results/page_test_results.py |
@@ -11,9 +11,22 @@ from telemetry import value as value_module |
from telemetry.value import failure |
class PageTestResults(object): |
- def __init__(self, output_stream=None, trace_tag=''): |
+ def __init__(self, output_stream=None, output_formatters=None, trace_tag=''): |
+ """ |
+ Args: |
+ output_stream: The output stream to use to write test results. |
+ output_formatters: A list of output formatters. The output |
+ formatters are typically used to format the test results, such |
+ as CsvOutputFormatter, which output the test results as CSV. |
+ trace_tag: A string to append to the buildbot trace |
+ name. Currently only used for buildbot. |
+ """ |
+ # TODO(chrishenry): Figure out if trace_tag is still necessary. |
+ |
super(PageTestResults, self).__init__() |
self._output_stream = output_stream |
+ self._output_formatters = ( |
+ output_formatters if output_formatters is not None else []) |
self._trace_tag = trace_tag |
self._current_page = None |
@@ -97,6 +110,9 @@ class PageTestResults(object): |
self.successes.append(page) |
def PrintSummary(self): |
+ for output_formatter in self._output_formatters: |
+ output_formatter.Format(self) |
+ |
if self.failures: |
logging.error('Failed pages:\n%s', '\n'.join( |
p.display_name for p in self.pages_that_had_failures)) |