Add cpuTimeMetric to BattOr benchmark.

tools/perf/benchmarks/battor.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from core import perf_benchmark
 from telemetry.timeline import chrome_trace_category_filter
 from telemetry.web_perf import timeline_based_measurement
 import page_sets
 from telemetry import benchmark
 
 
 # TODO(rnephew): Remove BattOr naming from all benchmarks once the BattOr tests
 # are the primary means of benchmarking power.
 class _BattOrBenchmark(perf_benchmark.PerfBenchmark):
 
   def CreateTimelineBasedMeasurementOptions(self):
-    options = timeline_based_measurement.Options(
-        chrome_trace_category_filter.ChromeTraceCategoryFilter())
+    category_filter = chrome_trace_category_filter.ChromeTraceCategoryFilter(
+        filter_string='toplevel')
+    options = timeline_based_measurement.Options(category_filter)
     options.config.chrome_trace_config.category_filter.AddFilterString('rail')
     options.config.enable_battor_trace = True
     options.config.enable_chrome_trace = True
     options.config.enable_atrace_trace = True
     options.config.atrace_config.categories = ['sched']
     options.config.enable_cpu_trace = True
-    options.SetTimelineBasedMetrics(['powerMetric', 'clockSyncLatencyMetric'])
+    options.SetTimelineBasedMetrics(
+        ['powerMetric', 'clockSyncLatencyMetric', 'cpuTimeMetric'])
     return options
 
   @classmethod
   def ShouldDisable(cls, possible_browser):
     # Only run if BattOr is detected.
     if not possible_browser.platform.HasBattOrConnected():
       return True
 
     # Galaxy S5s have problems with running system health metrics.
     # http://crbug.com/600463
@@ skipping 92 matching lines @@
     return page_sets.MacGpuTrivialPagesStorySet(wait_in_seconds=30)
 
   @classmethod
   def Name(cls):
     return 'battor.trivial_pages'
 
 @benchmark.Enabled('mac')
 class BattOrSteadyStatePages(_BattOrBenchmark):
 
   def CreateStorySet(self, options):
-    # We want it to wait for 30 seconds to be comparable to legacy power tests.
+    # We want it to wait for 30 seconds to be comparable to legacy power tests,
+    # but that produces too much tracing data. Instead wait for 15 seconds.

[charliea (OOO until 10-5), 2016/09/19 17:09:50]

nit: please eliminate the second line now that we just got rid of ABC news

[erikchen, 2016/09/19 17:19:38]

Done.

     return page_sets.IdleAfterLoadingStories(wait_in_seconds=30)
 
   @classmethod
   def Name(cls):
     return 'battor.steady_state'