Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(139)

Unified Diff: tools/perf/page_sets/system_health/loading_stories.py

Issue 2828403003: Remove less important system health mobile stories. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/perf/page_sets/system_health/browsing_stories.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/page_sets/system_health/loading_stories.py
diff --git a/tools/perf/page_sets/system_health/loading_stories.py b/tools/perf/page_sets/system_health/loading_stories.py
index 946e67db4f41a82b8fb7e300109c4013eb7c136e..b5c99a1cdb8b25581033d34283a3db45b60a9105 100644
--- a/tools/perf/page_sets/system_health/loading_stories.py
+++ b/tools/perf/page_sets/system_health/loading_stories.py
@@ -123,10 +123,11 @@ class LoadTumblrStory(_LoadingStory):
################################################################################
-class LoadBbcStory(_LoadingStory):
+class LoadBbcDesktopStory(_LoadingStory):
NAME = 'load:news:bbc'
# Redirects to the "http://" version.
URL = 'https://www.bbc.co.uk/news/world-asia-china-36189636'
+ SUPPORTED_PLATFORMS = platforms.DESKTOP_ONLY
class LoadCnnStory(_LoadingStory):
@@ -136,14 +137,16 @@ class LoadCnnStory(_LoadingStory):
TAGS = [story_tags.JAVASCRIPT_HEAVY]
-class LoadFlipboardStory(_LoadingStory):
+class LoadFlipboardDesktopStory(_LoadingStory):
NAME = 'load:news:flipboard'
URL = 'https://flipboard.com/explore'
+ SUPPORTED_PLATFORMS = platforms.DESKTOP_ONLY
-class LoadHackerNewsStory(_LoadingStory):
+class LoadHackerNewsDesktopStory(_LoadingStory):
NAME = 'load:news:hackernews'
URL = 'https://news.ycombinator.com'
+ SUPPORTED_PLATFORMS = platforms.DESKTOP_ONLY
class LoadNytimesDesktopStory(_LoadingStory):
@@ -177,18 +180,6 @@ class LoadRedditMobileStory(_LoadingStory):
SUPPORTED_PLATFORMS = platforms.MOBILE_ONLY
-class LoadSohuMobileStory(_LoadingStory):
- NAME = 'load:news:sohu'
- # Using "https://" leads to missing images and scripts on mobile (due to
- # mixed content).
- URL = 'http://m.sohu.com/n/447433356/'
- # The desktop page (http://news.sohu.com/20160503/n447433356.shtml) almost
- # always fails to completely load due to
- # https://github.com/chromium/web-page-replay/issues/74.
- SUPPORTED_PLATFORMS = platforms.MOBILE_ONLY
- TAGS = [story_tags.INTERNATIONAL]
-
-
class LoadWashingtonPostMobileStory(_LoadingStory):
NAME = 'load:news:washingtonpost'
URL = 'https://www.washingtonpost.com/pwa'
@@ -253,14 +244,16 @@ class LoadSoundCloudStory(_LoadingStory):
URL = 'https://soundcloud.com/lifeofdesiigner/desiigner-panda'
-class Load9GagStory(_LoadingStory):
+class Load9GagDesktopStory(_LoadingStory):
NAME = 'load:media:9gag'
URL = 'https://www.9gag.com/'
+ SUPPORTED_PLATFORMS = platforms.DESKTOP_ONLY
-class LoadFlickrStory(_LoadingStory):
+class LoadFlickrDesktopStory(_LoadingStory):
NAME = 'load:media:flickr'
URL = 'https://www.flickr.com/photos/tags/farm'
+ SUPPORTED_PLATFORMS = platforms.DESKTOP_ONLY
def _DidLoadDocument(self, action_runner):
# Wait until the 'Recently tagged' view loads.
« no previous file with comments | « tools/perf/page_sets/system_health/browsing_stories.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698