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

Unified Diff: build/util/lib/common/perf_tests_results_helper.py

Issue 24253006: Android / Telemetry: final cleanup on perf_tests_helper. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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: build/util/lib/common/perf_tests_results_helper.py
diff --git a/build/util/lib/common/perf_tests_results_helper.py b/build/util/lib/common/perf_tests_results_helper.py
index 733cbf91acb129ebb683a2e1a38a671364df1c58..cacf54ed2c2d2b715d92c07b8197c3e21f07d98a 100644
--- a/build/util/lib/common/perf_tests_results_helper.py
+++ b/build/util/lib/common/perf_tests_results_helper.py
@@ -134,11 +134,12 @@ def PrintPerfResult(measurement, trace, values, units,
# across different histograms.
assert len(values) == 1
value = values[0]
- output = '%s%s: %s= %s' % (
+ output = '%s%s: %s= %s %s' % (
RESULT_TYPES[result_type],
_EscapePerfResult(measurement),
trace_name,
- value)
+ value,
+ units)
frankf 2013/09/20 02:07:39 why didn't this include units before?
bulach 2013/09/20 08:23:46 bad timing :) ryan splitted this file (build/util/
avg, sd = GeomMeanAndStdDevFromHistogram(value)
if avg:

Powered by Google App Engine
This is Rietveld 408576698