Rename rasterize_and_record_micro.key_mobile_sites{_smooth,} and top_25{_smooth,}

tools/perf/benchmarks/rasterize_and_record_micro.py

 # Copyright 2013 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
 
 import ct_benchmarks_util
 from measurements import rasterize_and_record_micro
 import page_sets
 from telemetry import benchmark
@@ skipping 34 matching lines @@
 
 
 # RasterizeAndRecord disabled on mac because of crbug.com/350684.
 # RasterizeAndRecord disabled on windows because of crbug.com/338057.
 @benchmark.Disabled('mac', 'win',
                     'android')  # http://crbug.com/610018
 class RasterizeAndRecordMicroTop25(_RasterizeAndRecordMicro):
   """Measures rasterize and record performance on the top 25 web pages.
 
   http://www.chromium.org/developers/design-documents/rendering-benchmarks"""
   page_set = page_sets.Top25PageSet

[wkorman, 2016/10/04 18:31:02]

This seems to be similar -- not smooth here, but name below is '_smooth'.

[wkorman, 2016/10/04 21:06:02]

With vmpstr@ feedback I'm supportive of this change then if we switch this as
well, unless there's some reason we shouldn't?

 
   @classmethod
   def Name(cls):
     return 'rasterize_and_record_micro.top_25_smooth'
 
 
 @benchmark.Disabled('mac', 'win',
                     'android')  # http://crbug.com/531597
 class RasterizeAndRecordMicroKeyMobileSites(_RasterizeAndRecordMicro):
   """Measures rasterize and record performance on the key mobile sites.
 
   http://www.chromium.org/developers/design-documents/rendering-benchmarks"""
-  page_set = page_sets.KeyMobileSitesPageSet
+  page_set = page_sets.KeyMobileSitesSmoothPageSet

[wkorman, 2016/10/04 18:31:02]

The other possibility is that the name was a typo, and we intended to keep this
page set but not have '_smooth' in the name.

+vmpstr@ to weigh in.

[vmpstr, 2016/10/04 19:00:14]

I think the benchmark might predate "smooth" pagesets, so maybe we just missed
updating it.

 
   @classmethod
   def Name(cls):
     return 'rasterize_and_record_micro.key_mobile_sites_smooth'
 
 
 @benchmark.Disabled('mac', 'win', 'android') # http://crbug.com/610424
 class RasterizeAndRecordMicroKeySilkCases(_RasterizeAndRecordMicro):
   """Measures rasterize and record performance on the silk sites.
 
@@ skipping 50 matching lines @@
     _RasterizeAndRecordMicro.AddBenchmarkCommandLineArgs(parser)
     ct_benchmarks_util.AddBenchmarkCommandLineArgs(parser)
 
   @classmethod
   def ProcessCommandLineArgs(cls, parser, args):
     ct_benchmarks_util.ValidateCommandLineArgs(parser, args)
 
   def CreateStorySet(self, options):
     return page_sets.CTPageSet(
         options.urls_list, options.user_agent, options.archive_data_file)