| Index: tools/perf/metrics/cpu.py
|
| diff --git a/tools/perf/metrics/cpu.py b/tools/perf/metrics/cpu.py
|
| index 3d54a9040efca5586db0fdc4c488976a79c824b9..d0a03040f439246b258fa6ac3cb98c180d49f79b 100644
|
| --- a/tools/perf/metrics/cpu.py
|
| +++ b/tools/perf/metrics/cpu.py
|
| @@ -11,9 +11,9 @@ class CpuMetric(Metric):
|
|
|
| def __init__(self, browser):
|
| super(CpuMetric, self).__init__()
|
| - self._results = None
|
| self._browser = browser
|
| self._start_cpu = None
|
| + self._stop_cpu = None
|
|
|
| def DidStartBrowser(self, browser):
|
| # Save the browser object so that cpu_stats can be accessed later.
|
| @@ -24,16 +24,17 @@ class CpuMetric(Metric):
|
|
|
| def Stop(self, page, tab):
|
| assert self._start_cpu, 'Must call Start() first'
|
| - self._results = _SubtractCpuStats(self._browser.cpu_stats, self._start_cpu)
|
| + self._stop_cpu = self._browser.cpu_stats
|
|
|
| # Optional argument trace_name is not in base class Metric.
|
| # pylint: disable=W0221
|
| def AddResults(self, tab, results, trace_name='cpu_utilization'):
|
| - assert self._results, 'Must call Stop() first'
|
| + assert self._stop_cpu, 'Must call Stop() first'
|
| + cpu_stats = _SubtractCpuStats(self._stop_cpu, self._start_cpu)
|
| # Add a result for each process type.
|
| - for process_type in self._results:
|
| + for process_type in cpu_stats:
|
| trace_name_for_process = '%s_%s' % (trace_name, process_type.lower())
|
| - cpu_percent = 100 * self._results[process_type]
|
| + cpu_percent = 100 * cpu_stats[process_type]
|
| results.AddValue(scalar.ScalarValue(
|
| results.current_page, 'cpu_utilization.%s' % trace_name_for_process,
|
| '%', cpu_percent, important=False))
|
|
|