Chromium Code Reviews| Index: telemetry/telemetry/internal/results/html_output_formatter.py |
| diff --git a/telemetry/telemetry/internal/results/html_output_formatter.py b/telemetry/telemetry/internal/results/html_output_formatter.py |
| index f325ef28eb603c78ab6cb902173bd85d52e793ff..905594e5d305700bb272e303fb2f4eec960fb04a 100644 |
| --- a/telemetry/telemetry/internal/results/html_output_formatter.py |
| +++ b/telemetry/telemetry/internal/results/html_output_formatter.py |
| @@ -12,6 +12,7 @@ from py_utils import cloud_storage # pylint: disable=import-error |
| from telemetry.core import util |
| from telemetry.internal.results import chart_json_output_formatter |
| +from telemetry.internal.results import html2_output_formatter |
| from telemetry.internal.results import output_formatter |
| from telemetry import value as value_module |
| from telemetry.value import list_of_scalar_values |
| @@ -45,7 +46,7 @@ class HtmlOutputFormatter(output_formatter.OutputFormatter): |
| self._reset_results = reset_results |
| self._upload_results = upload_results |
| self._build_time = self._GetBuildTime() |
| - self._existing_results = self._ReadExistingResults(output_stream) |
| + self._existing_results = [] |
|
nednguyen
2016/10/25 17:52:25
nits: there is no need to keep this self._existing
|
| if results_label: |
| self._results_label = results_label |
| else: |
| @@ -157,6 +158,14 @@ class HtmlOutputFormatter(output_formatter.OutputFormatter): |
| return all_results |
| def Format(self, page_test_results): |
| + if page_test_results.value_set: |
| + html2_formatter = html2_output_formatter.Html2OutputFormatter( |
| + self._output_stream, self._reset_results, self._upload_results) |
| + html2_formatter.Format(page_test_results) |
| + return |
| + |
| + self._existing_results = self._ReadExistingResults(self._output_stream) |
| + |
| chart_json_dict = chart_json_output_formatter.ResultsAsChartDict( |
| self._metadata, page_test_results.all_page_specific_values, |
| page_test_results.all_summary_values) |