Outputting chart json for disabled tests.

telemetry/telemetry/internal/results/results_options.py

 # Copyright 2014 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.
 
 import codecs
 import optparse
 import os
 import sys
 import time
 
@@ skipping 94 matching lines @@
     return open(output_file, mode='w+')
 
 
 def _GetProgressReporter(output_skipped_tests_summary, suppress_gtest_report):
   if suppress_gtest_report:
     return progress_reporter.ProgressReporter()
 
   return gtest_progress_reporter.GTestProgressReporter(
       sys.stdout, output_skipped_tests_summary=output_skipped_tests_summary)
 
+def FormatDisabledResults(benchmark_metadata, options):
+  if not options.output_formats:
+    # We only output disabled results for chartjson
+    return
+
+  if 'chartjson' in options.output_formats:

[nednguyen, 2016/09/22 13:24:33]

We should use polymorphism to handle this instead

[eyaich1, 2016/09/26 14:23:09]

Done.

+    output_stream = _GetOutputStream('chartjson', options.output_dir)
+    chart_json_output_formatter.ChartJsonOutputFormatter(
+              output_stream, benchmark_metadata).FormatDisabled()
 
 def CreateResults(benchmark_metadata, options,
                   value_can_be_added_predicate=lambda v, is_first: True):
   """
   Args:
     options: Contains the options specified in AddResultsOptions.
   """
   if not options.output_formats:
     options.output_formats = [_OUTPUT_FORMAT_CHOICES[0]]
 
@@ skipping 43 matching lines @@
       output_formatters=output_formatters, progress_reporter=reporter,
       output_dir=options.output_dir,
       value_can_be_added_predicate=value_can_be_added_predicate)
 
   results.iteration_info.benchmark_name = benchmark_metadata.name
   results.iteration_info.benchmark_start_ms = time.time() * 1000.0
   if options.results_label:
     results.iteration_info.label = options.results_label
 
   return results