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

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

Issue 58423002: [Telemetry] Print trace counters with trace events. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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/timeline.py
diff --git a/tools/perf/metrics/timeline.py b/tools/perf/metrics/timeline.py
index 5aa68b71949ad9210e6aa2fea633b38f8cdc2469..2bcee7bca48777ef3130af9a554503c60387ce51 100644
--- a/tools/perf/metrics/timeline.py
+++ b/tools/perf/metrics/timeline.py
@@ -73,3 +73,9 @@ class TimelineMetric(Metric):
results.Add(event_name, 'ms', total)
results.Add(event_name + '_max', 'ms', biggest_jank)
results.Add(event_name + '_avg', 'ms', total / len(times))
+
+ counters_by_name = self._thread_for_tab.parent.counters
+ for counter_name, counter in counters_by_name.iteritems():
+ total = sum(counter.totals)
+ results.Add(counter_name, '', total)
ernstm 2013/11/04 22:25:22 Please use 'count' instead of '' for the unit in l
hartmanng 2013/11/05 14:28:45 Done.
+ results.Add(counter_name + '_avg', '', total / len(counter.totals))
« 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