Add blink_perf.paint_spinvalidation and blink_perf.svg_spinvalidation suites

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 0cfd03a39ae10b672d583451834af1b34b563c86..f210adcdef2d7366db88e80a76fe1ff14b64d16e 100644
--- a/tools/perf/benchmarks/blink_perf.py
+++ b/tools/perf/benchmarks/blink_perf.py
@@ -119,6 +119,14 @@ class _BlinkPerfMeasurement(legacy_page_test.LegacyPageTest):
     print log
 
 
+class _BlinkPerfMeasurementSPInvalidation(_BlinkPerfMeasurement):

[pdr., 2016/12/06 06:47:28]

It's really long but WDYT of changing this to:
_BlinkPerfMeasurementSlimmingPaintInvalidation

I think this will help reduce confusion if other people read this code and are
not familiar with our 'SP' names.

[Xianzhu, 2016/12/06 17:23:34]

Done.

+  def CustomizeBrowserOptions(self, options):
+    _BlinkPerfMeasurement.CustomizeBrowserOptions(self, options)
+    options.AppendExtraBrowserArgs([
+        '--enable-blink-features=SlimmingPaintInvalidation'
+    ])
+
+
 class _SharedPywebsocketPageState(shared_page_state.SharedPageState):
   """Runs a pywebsocket server."""
 
@@ -267,6 +275,15 @@ class BlinkPerfPaint(perf_benchmark.PerfBenchmark):
     return cls.IsSvelte(possible_browser)  # http://crbug.com/574483
 
 
+class BlinkPerfPaintSPInvalidation(BlinkPerfPaint):

[pdr., 2016/12/06 06:47:28]

Can you add a link to either of the design docs and a note that this can be
removed when slimming-paint-invalidation ships (here and below)?

I think this can be done with a python class comment:
  """The blink_perf.paint_slimmingpaintinvalidation tests measure paint time
  with the new paint invalidation system (see: https://goo.gl/eQczQW). These
  tests should be removed when slimming paint invalidation ships.
  """

[Xianzhu, 2016/12/06 17:23:34]

I put the following doc in _BlinkPerfMeasurementSlimmingPaintInvalidation:

"""Measures blink perf with the new paint invalidation system (see:
https://goo.gl/eQczQW). The benchmarks using this measurement should be
removed when slimming paint invalidation ships."""

+  tag = 'paint_spinvalidation'
+  test = _BlinkPerfMeasurementSPInvalidation
+
+  @classmethod
+  def Name(cls):
+    return 'blink_perf.paint_spinvalidation'
+
+
 @benchmark.Disabled('win')  # crbug.com/488493
 class BlinkPerfParser(perf_benchmark.PerfBenchmark):
   tag = 'parser'
@@ -294,6 +311,15 @@ class BlinkPerfSVG(perf_benchmark.PerfBenchmark):
     return CreateStorySetFromPath(path, SKIPPED_FILE)
 
 
+class BlinkPerfSVGSPInvalidation(BlinkPerfSVG):
+  tag = 'svg_spinvalidation'
+  test = _BlinkPerfMeasurementSPInvalidation
+
+  @classmethod
+  def Name(cls):
+    return 'blink_perf.svg_spinvalidation'
+
+
 class BlinkPerfShadowDOM(perf_benchmark.PerfBenchmark):
   tag = 'shadow_dom'
   test = _BlinkPerfMeasurement