diff options
author | tonyg@chromium.org <tonyg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-19 15:36:38 +0000 |
---|---|---|
committer | tonyg@chromium.org <tonyg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-19 15:36:38 +0000 |
commit | 3c5ba1d96fd7a5684dec3485ca150d1f89920d13 (patch) | |
tree | 0ddf340c3ed12510766f354336ed7250b2f34b3d /build | |
parent | eb207b075cde68bb2f0b32045dfd53ca925bd81f (diff) | |
download | chromium_src-3c5ba1d96fd7a5684dec3485ca150d1f89920d13.zip chromium_src-3c5ba1d96fd7a5684dec3485ca150d1f89920d13.tar.gz chromium_src-3c5ba1d96fd7a5684dec3485ca150d1f89920d13.tar.bz2 |
[Telemetry] Fix histogram naming.
trace_tags should show up on the trace names but not the chart names. Previously
we'd output:
HISTOGRAM V8.MemoryHeapSampleTotalUsed_foo: V8.MemoryHeapSampleTotalUsed_foo=
Now we output:
HISTOGRAM V8.MemoryHeapSampleTotalUsed: V8.MemoryHeapSampleTotalUsed_foo=
This prevents an explosion of charts as seen on:
http://build.chromium.org/f/chromium/perf/linux-release/moz/report.html?history=150&rev=-1&graph=V8.MemoryExternalFragmentationTotal_extcs1_0.5
BUG=196411
TEST=tools/perf/run_multipage_benchmarks --browser=system page_cycler tools/perf/page_sets/page_cycler/moz.json --output-trace-tag=_foo
NOTRY=True
Review URL: https://chromiumcodereview.appspot.com/12900005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189022 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'build')
-rw-r--r-- | build/android/pylib/perf_tests_helper.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/build/android/pylib/perf_tests_helper.py b/build/android/pylib/perf_tests_helper.py index 44397c0..5ae6dc7 100644 --- a/build/android/pylib/perf_tests_helper.py +++ b/build/android/pylib/perf_tests_helper.py @@ -110,11 +110,10 @@ def PrintPerfResult(measurement, trace, values, units, result_type='default', # across different histograms. assert len(values) == 1 value = values[0] - measurement += '.' + trace_name output = '%s%s: %s= %s' % ( RESULT_TYPES[result_type], _EscapePerfResult(measurement), - _EscapePerfResult(measurement), + trace_name, value) avg, sd = GeomMeanAndStdDevFromHistogram(value) |