[Telemetry] Create loading.desktop, a desktop analog for loading.mobile

tools/perf/benchmarks/loading.py

Index: tools/perf/benchmarks/loading.py
diff --git a/tools/perf/benchmarks/loading.py b/tools/perf/benchmarks/loading.py
index 7aacd00996a987a311aac9202d0bcc35c18fb139..0b0c6650cb1a95bd45afed08ed5297fb65e90cbc 100644
--- a/tools/perf/benchmarks/loading.py
+++ b/tools/perf/benchmarks/loading.py
@@ -17,17 +17,39 @@ class _LoadingBase(perf_benchmark.PerfBenchmark):
   """ A base class for loading benchmarks. """
 
   options = {'pageset_repeat': 2}
+  SUBNAME = None
 
   def CreateTimelineBasedMeasurementOptions(self):
     tbm_options = timeline_based_measurement.Options()
     page_cycler_v2.AugmentOptionsForLoadingMetrics(tbm_options)
     return tbm_options
 
+  @classmethod
+  def Name(cls):
+    assert cls.SUBNAME
+    return 'loading.%s' % cls.SUBNAME

[nednguyen, 2017/04/26 21:02:36]

its' better just uses the raw strings so people can code search for it.

[rnephew (Reviews Here), 2017/04/26 21:15:43]

Acknowledged.

+
+
+@benchmark.Disabled('android')
+@benchmark.Owner(emails=['kouhei@chormium.org', 'ksakamoto@chromium.org'])
+class LoadingDesktop(_LoadingBase):
+  """ A benchmark measuring loading performance of desktop sites. """
+  SUBNAME = 'desktop'
+
+  @classmethod
+  def ShouldDisable(cls, possible_browser):
+    return possible_browser.browser_type == 'reference'
+
+  def CreateStorySet(self, options):
+    return page_sets.LoadingDesktopStorySet(
+        cache_temperatures=[cache_temperature.ANY])

[nednguyen, 2017/04/26 21:02:36]

I think we would need cold & warm cache 

[rnephew (Reviews Here), 2017/04/26 21:15:43]

The existing loading.mobile benchmark uses the same cache setting as this does.
Is there a reason we would want it here and not there?

[nednguyen, 2017/05/01 19:50:47]

Not sure, but for now, we need to make sure that this matches the existing
page_cycler_v2 benchmarks, which include:
pageset_repeat = 2
cache_temperatures=[
          cache_temperature.PCV1_COLD, cache_temperature.PCV1_WARM]

(https://cs.chromium.org/chromium/src/tools/perf/benchmarks/page_cycler_v2.py?...)

+
 
 @benchmark.Enabled('android')
 @benchmark.Owner(emails=['kouhei@chromium.org', 'ksakamoto@chromium.org'])
 class LoadingMobile(_LoadingBase):
   """ A benchmark measuring loading performance of mobile sites. """
+  SUBNAME = 'mobile'
 
   @classmethod
   def ShouldDisable(cls, possible_browser):
@@ -43,10 +65,6 @@ class LoadingMobile(_LoadingBase):
 
     return False
 
-  @classmethod
-  def Name(cls):
-    return 'loading.mobile'
-
   def CreateStorySet(self, options):
     return page_sets.LoadingMobileStorySet(
         cache_temperatures=[cache_temperature.ANY],
@@ -62,9 +80,8 @@ class LoadingClusterTelemetry(_LoadingBase):
 
   _ALL_NET_CONFIGS = traffic_setting.NETWORK_CONFIGS.keys()
 
-  @classmethod
-  def Name(cls):
-    return 'loading.cluster_telemetry'
+  SUBNAME = 'cluster_telemetry'
+
 
   @classmethod
   def AddBenchmarkCommandLineArgs(cls, parser):