Add WebView startup time benchmark

tools/perf/benchmarks/system_health.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 import benchmark
 from telemetry.timeline import tracing_category_filter
 from telemetry.web_perf import timeline_based_measurement
 import page_sets
 
@@ skipping 20 matching lines @@
 
 
 @benchmark.Disabled('all')  # crbug.com/613050
 class SystemHealthTop25(_SystemHealthBenchmark):
   page_set = page_sets.Top25PageSet
 
   @classmethod
   def Name(cls):
     return 'system_health.top25'
 
+
 @benchmark.Disabled('android')  # crbug.com/601953
 @benchmark.Disabled('all')  # crbug.com/613050
 class SystemHealthKeyMobileSites(_SystemHealthBenchmark):
   page_set = page_sets.KeyMobileSitesPageSet
 
   @classmethod
   def Name(cls):
     return 'system_health.key_mobile_sites'
 
 
@@ skipping 15 matching lines @@
         tracing_category_filter.TracingCategoryFilter(
             '-*,disabled-by-default-memory-infra'))
     options.config.enable_android_graphics_memtrack = True
     options.SetTimelineBasedMetric('memoryMetric')
     return options
 
   @classmethod
   def Name(cls):
     return 'system_health.memory_%s' % cls.page_set.PLATFORM
 
+@benchmark.Enabled('android')
+class WebviewStartupSystemHealthBenchmark(_SystemHealthBenchmark):
+  """Chrome Webview Startup Time System Health Benchmark.

[charliea (OOO until 10-5), 2016/06/08 20:57:58]

I don't think this first line is useful. I'd prefer just 

"""A benchmark that measures how long it takes WebView to start up and load a
blank page.
"""

[alexandermont, 2016/06/08 21:09:15]

Done

+
+  Benchmark that measures the amount of time that WebView takes
+  to start up and then to load a blank page.
+  """
+  page_set = page_sets.BlankPageSet

[charliea (OOO until 10-5), 2016/06/08 20:57:58]

nit: can you add a blank line between this line and the next?

[alexandermont, 2016/06/08 21:09:15]

Done

+  def CreateTimelineBasedMeasurementOptions(self):
+    options = timeline_based_measurement.Options(

[charliea (OOO until 10-5), 2016/06/08 20:57:58]

nit: this can be deleted (it's the default)

[alexandermont, 2016/06/08 21:09:15]

Done

+        tracing_category_filter.TracingCategoryFilter())
+    options.SetTimelineBasedMetric('webviewStartupMetric')
+    options.config.enable_atrace_trace = True
+    options.config.enable_chrome_trace = False

[charliea (OOO until 10-5), 2016/06/08 20:57:58]

nit: this can be deleted (it's the default)

[alexandermont, 2016/06/08 21:09:15]

Done

+    options.config.atrace_config.app_name = 'org.chromium.webview_shell'
+    return options
+
+  @classmethod
+  def Name(cls):
+    return 'system_health.webview_startup'
+
 # https://github.com/catapult-project/catapult/issues/2340
 @benchmark.Disabled('all')
 class DesktopMemorySystemHealth(_MemorySystemHealthBenchmark):
   """Desktop Chrome Memory System Health Benchmark."""
   page_set = page_sets.DesktopMemorySystemHealthStorySet
 
   @classmethod
   def ShouldDisable(cls, possible_browser):
     return possible_browser.platform.GetDeviceTypeName() != 'Desktop'
 
 # https://github.com/catapult-project/catapult/issues/2340
 @benchmark.Disabled('all')
 class MobileMemorySystemHealth(_MemorySystemHealthBenchmark):
   """Mobile Chrome Memory System Health Benchmark."""
   page_set = page_sets.MobileMemorySystemHealthStorySet
 
   @classmethod
   def ShouldDisable(cls, possible_browser):
     # http://crbug.com/612144 (reference on Nexus 5X).
     return possible_browser.platform.GetDeviceTypeName() == 'Desktop' or (
         possible_browser.browser_type == 'reference' and
         possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X')