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

Unified Diff: tools/perf/metrics/cpu.py

Issue 184713008: Telemetry CPU metric trace names are misleading (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 6 years, 10 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/metrics/cpu.py
diff --git a/tools/perf/metrics/cpu.py b/tools/perf/metrics/cpu.py
index a00c295e4762e3591bc4add87586ba4404704c3f..aa57b073078a791ea945e954f030c1c3cf9b3d9a 100644
--- a/tools/perf/metrics/cpu.py
+++ b/tools/perf/metrics/cpu.py
@@ -30,10 +30,10 @@ class CpuMetric(Metric):
assert self._results, 'Must call Stop() first'
# Add a result for each process type.
for process_type in self._results:
- trace_name = '%s_%s' % (trace_name, process_type.lower())
+ trace_name_for_process = '%s_%s' % (trace_name, process_type.lower())
cpu_percent = 100 * self._results[process_type]
- results.Add(trace_name, '%', cpu_percent, chart_name='cpu_utilization',
- data_type='unimportant')
+ results.Add(trace_name_for_process, '%', cpu_percent,
+ chart_name='cpu_utilization', data_type='unimportant')
def _SubtractCpuStats(cpu_stats, start_cpu_stats):
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698