New page set added to include pathologically bad and janky sites.

tools/perf/page_sets/pathological_mobile_sites.py

+# Copyright 2015 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 telemetry.page import page as page_module
+from telemetry.page import page_set as page_set_module
+
+
+class PathologicalMobileSitesPage(page_module.Page):
+
+  def __init__(self, url, page_set):
+    super(PathologicalMobileSitesPage, self).__init__(
+        url=url, page_set=page_set, credentials_path='data/credentials.json')
+    self.user_agent_type = 'mobile'
+    self.archive_data_file = 'data/pathological_mobile_sites.json'
+
+  def RunPageInteractions(self, action_runner):
+    interaction = action_runner.BeginGestureInteraction(
+        'ScrollAction', is_smooth=True)
+    action_runner.ScrollPage()
+    interaction.End()
+
+
+class PathologicalMobileSitesPageSet(page_set_module.PageSet):

[jdduke (slow), 2015/01/28 17:46:52]

Do we want to add this as a smoothness test?

[picksi, 2015/02/10 18:10:46]

Not sure what you want here. Do I need to add something somewhere else to make
this a "smoothness test", or are you suggesting I change is_smooth to False?
Happy to do whatever you are after!

[jdduke (slow), 2015/02/10 18:17:06]

Sorry, I meant add it to the smoothness suite of tests in "smoothness.py", much
as you added it to the "task_execution_time.py" suite. We're as concerned about
latency and jank here as we are about the task execution times, right?

+
+  """Pathologically bad and janky sites on mobile."""
+
+  def __init__(self):
+    super(PathologicalMobileSitesPageSet, self).__init__(
+        user_agent_type='mobile',
+        archive_data_file='data/pathological_mobile_sites.json',
+        bucket=page_set_module.INTERNAL_BUCKET)
+
+    sites = ['http://edition.cnn.com',
+             'http://m.espn.go.com/nhl/rankings',
+             'http://recode.net',
+             'http://www.forbes.com',
+             'http://www.latimes.com',
+             'http://www.pbs.org/newshour/bb/'
+             'much-really-cost-live-city-like-seattle/#the-rundown',

[Sami, 2015/01/27 15:30:37]

Please indent this line or use braces to make it more obvious that it's a
continuation of the previous one.

[picksi, 2015/02/10 18:10:46]

Done.

+             'http://www.zdnet.com']
+
+    for site in sites:
+      self.AddUserStory(PathologicalMobileSitesPage(site, self))