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

Unified Diff: tools/perf/measurements/page_cycler.py

Issue 239083010: Telemetry: adds CPU frequency stats. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comments 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
« no previous file with comments | « no previous file | tools/perf/metrics/cpu.py » ('j') | tools/perf/metrics/cpu.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/measurements/page_cycler.py
diff --git a/tools/perf/measurements/page_cycler.py b/tools/perf/measurements/page_cycler.py
index b7dffbc461c223fda18aeae8f1b6eb032e1a1c3b..e7803a41844d985872dfb74f5525e09f3dd3b53b 100644
--- a/tools/perf/measurements/page_cycler.py
+++ b/tools/perf/measurements/page_cycler.py
@@ -81,15 +81,11 @@ class PageCycler(page_measurement.PageMeasurement):
tab.ClearCache(force=True)
if self._report_speed_index:
self._speedindex_metric.Start(page, tab)
+ self._cpu_metric.Start(page, tab)
def DidNavigateToPage(self, page, tab):
self._memory_metric.Start(page, tab)
self._power_metric.Start(page, tab)
- # TODO(qyearsley): Uncomment the following line and move it to
- # WillNavigateToPage once the cpu metric has been changed.
- # This is being temporarily commented out to let the page cycler
- # results return to how they were before the cpu metric was added.
- # self._cpu_metric.Start(page, tab) See crbug.com/301714.
if self._record_v8_object_stats:
self._v8_object_stats_metric.Start(page, tab)
@@ -144,8 +140,8 @@ class PageCycler(page_measurement.PageMeasurement):
# TODO(qyearsley): Uncomment the following line when CPU metric is
# changed. See crbug.com/301714.
qyearsley 2014/04/22 18:22:26 This TODO comment can be removed as well, right?
bulach 2014/04/22 18:34:50 this patch is not yet ready to land :) I'll merge
- # self._cpu_metric.Stop(page, tab)
- # self._cpu_metric.AddResults(tab, results)
+ self._cpu_metric.Stop(page, tab)
+ self._cpu_metric.AddResults(tab, results)
if self._record_v8_object_stats:
self._v8_object_stats_metric.Stop(page, tab)
self._v8_object_stats_metric.AddResults(tab, results)
« no previous file with comments | « no previous file | tools/perf/metrics/cpu.py » ('j') | tools/perf/metrics/cpu.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698