Fix page interaction with CnnArticlePage

tools/perf/page_sets/key_mobile_sites_smooth.py

Index: tools/perf/page_sets/key_mobile_sites_smooth.py
diff --git a/tools/perf/page_sets/key_mobile_sites_smooth.py b/tools/perf/page_sets/key_mobile_sites_smooth.py
index 200915991c0fa738fa683a25019fca8383ac6697..34baff9b55e08feb06ab87b3a8df10a1d9c1f9e9 100644
--- a/tools/perf/page_sets/key_mobile_sites_smooth.py
+++ b/tools/perf/page_sets/key_mobile_sites_smooth.py
@@ -125,6 +125,15 @@ class AmazonNicolasCageSmoothPage(
           selector='#search',
           distance_expr='document.body.scrollHeight - window.innerHeight')
 
+class CNNArticleSmoothPage(
+      key_mobile_sites_pages.CnnArticlePage):
+
+  def RunPageInteractions(self, action_runner):
+    with action_runner.CreateGestureInteraction('ScrollAction'):
+      action_runner.ScrollElement(
+          element_function='document.getElementsByTagName("body")[0]',
+          distance_expr='document.body.scrollHeight - window.innerHeight',
+          top_start_ratio=0.01)

[sullivan, 2016/08/24 14:40:03]

Reading the bug, we need to do this because the default scroll action is trying
to scroll an input element? Would be awesome to add some comments explaining why
we special-case this page in case anyone needs to understand it in the future.

[Navid Zolghadr, 2016/08/24 15:05:15]

Done.

 
 class KeyMobileSitesSmoothPageSet(story.StorySet):
 
@@ -140,7 +149,6 @@ class KeyMobileSitesSmoothPageSet(story.StorySet):
     predefined_page_classes = [
       key_mobile_sites_pages.CapitolVolkswagenPage,
       key_mobile_sites_pages.TheVergeArticlePage,
-      key_mobile_sites_pages.CnnArticlePage,
       key_mobile_sites_pages.FacebookPage,
       key_mobile_sites_pages.YoutubeMobilePage,
       key_mobile_sites_pages.YahooAnswersPage,
@@ -164,6 +172,7 @@ class KeyMobileSitesSmoothPageSet(story.StorySet):
     # self.AddStory(GroupClonedListImagesSmoothPage(self))
     self.AddStory(GoogleNewsMobile2SmoothPage(self))
     self.AddStory(AmazonNicolasCageSmoothPage(self))
+    self.AddStory(CNNArticleSmoothPage(self))
 
     # Add pages with custom labels.