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

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

Issue 239083010: Telemetry: adds CPU frequency stats. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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: tools/perf/metrics/cpu_unittest.py
diff --git a/tools/perf/metrics/cpu_unittest.py b/tools/perf/metrics/cpu_unittest.py
index 30d488dac8bc420399e00f32ea8a6fff6b8277d0..0b5b14b5d194fa7037c12b7d8399b3ef0d4391aa 100644
--- a/tools/perf/metrics/cpu_unittest.py
+++ b/tools/perf/metrics/cpu_unittest.py
@@ -31,3 +31,18 @@ class CpuMetricTest(unittest.TestCase):
self.assertEqual({'Browser': 0.25, 'Renderer': 0.1},
cpu._SubtractCpuStats(end, start))
+ # The result for global frequency stats is used.
+ start = {
+ 'Browser': {'CpuProcessTime': 0, 'TotalTime': 0},
+ 'Global': {'GlobalCpuFrequencyStats': {'1_hz': 9, '2_hz': 19}}
+ }
+ end = {
+ 'Browser': {'CpuProcessTime': 5, 'TotalTime': 20},
+ 'Global': {'GlobalCpuFrequencyStats': {'1_hz': 20, '2_hz': 31}}
+ }
+
+ expected = {
+ 'Browser': 0.25,
+ 'Global': {'GlobalCpuFrequencyStats': {'1_hz': 11, '2_hz': 12}}
+ }
+ self.assertEqual(expected, cpu._SubtractCpuStats(end, start))

Powered by Google App Engine
This is Rietveld 408576698