[system health] Fix load:news:washingtonpost and browse:news:washingtonpost on Nexus9.

tools/perf/page_sets/system_health/browsing_stories.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.
 
 from page_sets.system_health import platforms
 from page_sets.system_health import system_health_story
 
 
 class _BrowsingStory(system_health_story.SystemHealthStory):
   """Abstract base class for browsing stories.
@@ skipping 164 matching lines @@
 
 class WashingtonPostMobileStory(_NewsBrowsingStory):
   """Progressive website"""
   NAME = 'browse:news:washingtonpost'
   URL = 'https://www.washingtonpost.com/pwa'
   IS_SINGLE_PAGE_APP = True
   ITEM_SELECTOR = '.hed > a'
   SUPPORTED_PLATFORMS = platforms.MOBILE_ONLY
 
   def _DidLoadDocument(self, action_runner):
-    # Close the popup window.
-    action_runner.ClickElement(selector='.close')
+    # Close the popup window. On Nexus 9 (and probably other tables) the popup
+    # window does not have a "Close" button, instead it has only a "Send link
+    # to phone" button. So on tablets we run with the popup window open. The
+    # popup is transparent, so this is mostly an aesthetical issue.
+    buttonSelector = '.close'

[petrcermak, 2016/07/19 17:41:24]

button_selector. Furthermore, this could be a class constant
(_CLOSE_BUTTON_SELECTOR)

[ulan, 2016/07/19 17:48:55]

Done.

+    hasButton = action_runner.EvaluateJavaScript(

[petrcermak, 2016/07/19 17:41:24]

has_button

[ulan, 2016/07/19 17:48:55]

Done.

+        '!!document.querySelector("%s")' % buttonSelector)
+    if hasButton:
+      action_runner.ClickElement(selector=buttonSelector)
     super(WashingtonPostMobileStory, self)._DidLoadDocument(action_runner)