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

Unified Diff: tools/telemetry/telemetry/core/browser.py

Issue 239083010: Telemetry: adds CPU frequency stats. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes mac Created 6 years, 7 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 | « tools/perf/metrics/cpu_unittest.py ('k') | tools/telemetry/telemetry/core/platform/platform_backend.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/telemetry/telemetry/core/browser.py
diff --git a/tools/telemetry/telemetry/core/browser.py b/tools/telemetry/telemetry/core/browser.py
index 39a18cfb30001a042e7df8c2400b1d677e9f75cd..ff015365045c009fadbd85eac9d55a86d74d2e9d 100644
--- a/tools/telemetry/telemetry/core/browser.py
+++ b/tools/telemetry/telemetry/core/browser.py
@@ -203,6 +203,10 @@ class Browser(object):
'CpuProcessTime': S,
'TotalTime': T
}
+ 'Global': {
+ 'GlobalCpuFrequencyStats': {
+ },
+ }
}
Any of the above keys may be missing on a per-platform basis.
"""
@@ -216,6 +220,11 @@ class Browser(object):
if not len(result[process_type]):
continue
result[process_type].update(cpu_timestamp)
+
+ # Fetch system-wide CPU stats as well.
+ global_cpu_stats = self._platform_backend.GetGlobalCpuStats()
+ if global_cpu_stats:
+ result['Global'] = global_cpu_stats
return result
@property
« no previous file with comments | « tools/perf/metrics/cpu_unittest.py ('k') | tools/telemetry/telemetry/core/platform/platform_backend.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698