[PCv2] migrate page_cycler_v2.intl_ja_zh

tools/perf/benchmarks/page_cycler_v2.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.
 
 """The page cycler v2.
 
 For details, see design doc:
 https://docs.google.com/document/d/1EZQX-x3eEphXupiX-Hq7T4Afju5_sIdxPWYetj7ynd0
 """
 
 from core import perf_benchmark
 import page_sets
 
 from telemetry import benchmark
 from telemetry.page import cache_temperature
 from telemetry.timeline import tracing_category_filter
 from telemetry.web_perf import timeline_based_measurement
 

[nednguyen, 2016/06/27 04:33:48]

nits: 2 blank lines between top level definitions

[kouhei (in TOK), 2016/06/27 04:37:01]

Done.

-
+# crbug.com/619254
+@benchmark.Disabled('reference')
 class _PageCyclerV2(perf_benchmark.PerfBenchmark):
   def CreateTimelineBasedMeasurementOptions(self):
     cat_filter = tracing_category_filter.TracingCategoryFilter(
         filter_string='blink.console,navigation,blink.user_timing,loading')
 
     # Below categories are needed for first-meaningful-paint computation.
     cat_filter.AddDisabledByDefault('disabled-by-default-blink.debug.layout')
     cat_filter.AddIncludedCategory('devtools.timeline')
 
     tbm_options = timeline_based_measurement.Options(
         overhead_level=cat_filter)
     tbm_options.SetTimelineBasedMetric('firstPaintMetric')
     return tbm_options
 
-# crbug.com/619254
-@benchmark.Disabled('reference')
+  @classmethod
+  def ShouldDisable(cls, possible_browser):
+    # crbug.com/616781
+    if (cls.IsSvelte(possible_browser) or
+        possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X' or
+        possible_browser.platform.GetDeviceTypeName() == 'AOSP on BullHead'):
+      return True
+    return False
+

[nednguyen, 2016/06/27 04:33:48]

Same here

[kouhei (in TOK), 2016/06/27 04:37:01]

Done.

 class PageCyclerV2Typical25(_PageCyclerV2):
   """Page load time benchmark for a 25 typical web pages.
 
   Designed to represent typical, not highly optimized or highly popular web
   sites. Runs against pages recorded in June, 2014.
   """
   options = {'pageset_repeat': 2}
 
   @classmethod
   def Name(cls):
     return 'page_cycler_v2.typical_25'
 
-  @classmethod
-  def ShouldDisable(cls, possible_browser):
-    # crbug.com/616781
-    if (cls.IsSvelte(possible_browser) or
-        possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X' or
-        possible_browser.platform.GetDeviceTypeName() == 'AOSP on BullHead'):
-      return True
-    return False
-
   def CreateStorySet(self, options):
     return page_sets.Typical25PageSet(run_no_page_interactions=True,
         cache_temperatures=[
           cache_temperature.PCV1_COLD, cache_temperature.PCV1_WARM])
+

[nednguyen, 2016/06/27 04:33:48]

ditto

[kouhei (in TOK), 2016/06/27 04:37:01]

Done.

+class PageCyclerIntlJaZh(_PageCyclerV2):

[nednguyen, 2016/06/27 04:33:48]

You will want to make sure that this is named PageCyclerV2IntlJaZh, otherwise
telemetry code that discovers benchmarks get confused since it discovers all
classes indexed by their names.

I will add a test coverage to make sure that no two benchmark has same class
names due to this limitation.

[kouhei (in TOK), 2016/06/27 04:37:01]

Done.

+  """Page load time benchmark for a variety of pages in Japanese and Chinese.
+
+  Runs against pages recorded in April, 2013.
+  """
+  page_set = page_sets.IntlJaZhPageSet
+
+  @classmethod
+  def Name(cls):
+    return 'page_cycler_v2.intl_ja_zh'
+
+  def CreateStorySet(self, options):
+    return page_sets.IntlJaZhPageSet(cache_temperatures=[
+          cache_temperature.PCV1_COLD, cache_temperature.PCV1_WARM])