Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(201)

Unified Diff: tools/telemetry/telemetry/results/results_options.py

Issue 411143003: Add an "in_unit_test_mode" attribute to the environment. Use this to default output format to 'gtes… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comment. Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/telemetry/telemetry/results/results_options.py
diff --git a/tools/telemetry/telemetry/results/results_options.py b/tools/telemetry/telemetry/results/results_options.py
index d64c4e691f3d0a65350832f6e8bdffcb2c0bd0a7..83acae152782fca4a13cd9a96540c45d90df9b0a 100644
--- a/tools/telemetry/telemetry/results/results_options.py
+++ b/tools/telemetry/telemetry/results/results_options.py
@@ -7,7 +7,6 @@ import os
import sys
from telemetry.core import util
-from telemetry.page import page_measurement
from telemetry.results import block_page_measurement_results
from telemetry.results import buildbot_page_measurement_results
from telemetry.results import csv_page_measurement_results
@@ -44,11 +43,6 @@ def AddResultsOptions(parser):
def PrepareResults(test, options):
- if not isinstance(test, page_measurement.PageMeasurement):
- # Sort of hacky. The default for non-Measurements should be "gtest."
- if options.output_format != 'none':
- options.output_format = 'gtest'
-
if options.output_format == 'html' and not options.output_file:
options.output_file = os.path.join(util.GetBaseDir(), 'results.html')

Powered by Google App Engine
This is Rietveld 408576698