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

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

Issue 144543007: [Telemetry] Add power usage metric (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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/power.py » ('j') | tools/perf/metrics/power.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/measurements/startup.py
diff --git a/tools/perf/measurements/startup.py b/tools/perf/measurements/startup.py
index c9bb6b30a9dacc39e1ca465e70648b3ba982e60c..8831327381c53af68eb1901f56892cb4b1aef018 100644
--- a/tools/perf/measurements/startup.py
+++ b/tools/perf/measurements/startup.py
@@ -2,6 +2,7 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+from metrics import power
from metrics import startup_metric
from telemetry.page import page_measurement
@@ -17,6 +18,7 @@ class Startup(page_measurement.PageMeasurement):
def __init__(self, action_name_to_run = ''):
super(Startup, self).__init__(needs_browser_restart_after_each_run=True,
action_name_to_run=action_name_to_run)
+ self._power_metric = None
tonyg 2014/01/23 18:23:40 startup represents such a fleeting period that it
jeremy 2014/01/23 18:58:41 Done.
def AddCommandLineOptions(self, parser):
parser.add_option('--cold', action='store_true',
@@ -43,7 +45,13 @@ class Startup(page_measurement.PageMeasurement):
# Overriden so that no page navigation occurs - startup to the NTP.
pass
+ def WillStartBrowser(self, browser):
+ self._power_metric = power.PowerMetric()
+ self._power_metric.Start(None, None)
+
def MeasurePage(self, page, tab, results):
+ self._power_metric.Stop(None, None)
+ self._power_metric.AddResults(tab, results)
startup_metric.StartupMetric().AddResults(tab, results)
@@ -60,4 +68,4 @@ class StartWithUrl(Startup):
"""
def __init__(self):
- super(StartWithUrl, self).__init__(action_name_to_run='navigate_steps')
+ super(StartWithUrl, self).__init__(action_name_to_run='navigate_steps')
« no previous file with comments | « no previous file | tools/perf/metrics/power.py » ('j') | tools/perf/metrics/power.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698