[telemetry] Split blink_perf up on Android.

tools/perf/benchmarks/blink_perf.py

Index: tools/perf/benchmarks/blink_perf.py
diff --git a/tools/perf/benchmarks/blink_perf.py b/tools/perf/benchmarks/blink_perf.py
index 479986dea3f330ebecee1f1fe01266b9411a2e1e..8aba53471027a6e166b006020ea7e4fa390df80e 100644
--- a/tools/perf/benchmarks/blink_perf.py
+++ b/tools/perf/benchmarks/blink_perf.py
@@ -93,6 +93,7 @@ class _BlinkPerfMeasurement(page_test.PageTest):
     print log
 
 
+@benchmark.Disabled('android')

[tonyg, 2014/09/12 01:09:09]

Please add a TODO indicating that once the desktop bots are running on recipes
we should remove this benchmark and enable the below on all platforms.

[dtu, 2014/09/12 03:18:52]

Done.

 class BlinkPerfAll(benchmark.Benchmark):
   tag = 'all'
   test = _BlinkPerfMeasurement
@@ -103,7 +104,7 @@ class BlinkPerfAll(benchmark.Benchmark):
     return _CreatePageSetFromPath(path)
 
 
-@benchmark.Disabled
+@benchmark.Enabled('android')
 class BlinkPerfAnimation(benchmark.Benchmark):
   tag = 'animation'
   test = _BlinkPerfMeasurement
@@ -112,3 +113,135 @@ class BlinkPerfAnimation(benchmark.Benchmark):
     path = os.path.join(util.GetChromiumSrcDir(),
         'third_party', 'WebKit', 'PerformanceTests', 'Animation')
     return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfBindings(benchmark.Benchmark):
+  tag = 'bindings'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Bindings')

[tonyg, 2014/09/12 01:09:09]

I'm on the fence about whether these dir names should be dynamic or static. Just
curious what the pros/cons are.

[dtu, 2014/09/12 03:18:52]

Pros:
- Much shorter code. Easy to glance at list of tests.
- Adding or removing dirs will automatically add or remove tests.

Cons:
- More complex code, possibly involving inspect.
- Hard to @Disable individual dirs.
- I excluded some folders that will always be skipped. (Dromaeo, Replay,
SunSpider, XSSAuditor, inspector) We'd need a blacklist here to avoid extra test
steps on the waterfall.

[tonyg, 2014/09/12 14:12:50]

Makes sense. I agree with your approach.

+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfCSS(benchmark.Benchmark):
+  tag = 'css'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'CSS')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfCanvas(benchmark.Benchmark):
+  tag = 'canvas'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Canvas')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfDOM(benchmark.Benchmark):
+  tag = 'dom'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'DOM')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfEvents(benchmark.Benchmark):
+  tag = 'events'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Events')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfInteractive(benchmark.Benchmark):
+  tag = 'interactive'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Interactive')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfLayout(benchmark.Benchmark):
+  tag = 'layout'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Layout')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfMutation(benchmark.Benchmark):
+  tag = 'mutation'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Mutation')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfParser(benchmark.Benchmark):
+  tag = 'parser'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'Parser')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfSVG(benchmark.Benchmark):
+  tag = 'svg'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'SVG')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfShadowDOM(benchmark.Benchmark):
+  tag = 'shadow_dom'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'ShadowDOM')
+    return _CreatePageSetFromPath(path)
+
+
+@benchmark.Enabled('android')
+class BlinkPerfXMLHttpRequest(benchmark.Benchmark):
+  tag = 'xml_http_request'
+  test = _BlinkPerfMeasurement
+
+  def CreatePageSet(self, options):
+    path = os.path.join(util.GetChromiumSrcDir(),
+        'third_party', 'WebKit', 'PerformanceTests', 'XMLHttpRequest')
+    return _CreatePageSetFromPath(path)