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

Unified Diff: tools/perf/page_sets/key_mobile_sites_smooth.py

Issue 959063002: key_mobile_sites_smooth: Reload pages before scrolling for LinkedIn and Wowwiki (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: reload page before scrolling Created 5 years, 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 918e46ddcddf911e16e733b5e51637addf9aa090..92f5f28421040c62ee370feb734c8b6fddcf0982 100644
--- a/tools/perf/page_sets/key_mobile_sites_smooth.py
+++ b/tools/perf/page_sets/key_mobile_sites_smooth.py
@@ -40,6 +40,39 @@ class KeyMobileSitesSmoothPage(page_module.Page):
_IssueMarkerAndScroll(action_runner)
+class LinkedInSmoothPage(key_mobile_sites_pages.LinkedInPage):
+
+ def __init__(self, page_set):
+ super(LinkedInSmoothPage, self).__init__(page_set=page_set)
+
+ # Linkedin has expensive shader compilation so it can benefit from shader
+ # cache from reload.
+ def RunNavigateSteps(self, action_runner):
+ super(LinkedInSmoothPage, self).RunNavigateSteps(action_runner)
+ # Give it time to build cache.
+ action_runner.Wait(5)
nednguyen 2015/03/03 17:10:10 I don't like random wait time. Please find another
Yufeng Shen (Slow to review) 2015/03/03 19:03:28 Done.
+ action_runner.ReloadPage()
+ super(LinkedInSmoothPage, self).RunNavigateSteps(action_runner)
+
+
+# Why: Mobile wiki
nednguyen 2015/03/03 17:10:10 Make this comment the class's docstring
Yufeng Shen (Slow to review) 2015/03/03 19:03:28 Done.
+class WowwikiSmoothPage(KeyMobileSitesSmoothPage):
+
+ def __init__(self, page_set):
+ super(WowwikiSmoothPage, self).__init__(
+ url='http://www.wowwiki.com/World_of_Warcraft:_Mists_of_Pandaria',
+ page_set=page_set)
+
+ # Wowwiki has expensive shader compilation so it can benefit from shader
+ # cache from reload.
+ def RunNavigateSteps(self, action_runner):
+ super(WowwikiSmoothPage, self).RunNavigateSteps(action_runner)
+ # Give it time to build cache.
+ action_runner.Wait(5)
+ action_runner.ReloadPage()
+ super(WowwikiSmoothPage, self).RunNavigateSteps(action_runner)
+
+
class GmailSmoothPage(key_mobile_sites_pages.GmailPage):
def RunPageInteractions(self, action_runner):
@@ -129,7 +162,6 @@ class KeyMobileSitesSmoothPageSet(page_set_module.PageSet):
key_mobile_sites_pages.CnnArticlePage,
key_mobile_sites_pages.FacebookPage,
key_mobile_sites_pages.YoutubeMobilePage,
- key_mobile_sites_pages.LinkedInPage,
key_mobile_sites_pages.YahooAnswersPage,
key_mobile_sites_pages.GoogleNewsMobilePage,
]
@@ -137,6 +169,10 @@ class KeyMobileSitesSmoothPageSet(page_set_module.PageSet):
self.AddUserStory(
_CreatePageClassWithSmoothInteractions(page_class)(self))
+ self.AddUserStory(
+ _CreatePageClassWithSmoothInteractions(LinkedInSmoothPage)(self))
+ self.AddUserStory(WowwikiSmoothPage(self))
+
# Add pages with custom page interaction logic.
# Page behaves non-deterministically, replaced with test version for now.
@@ -253,8 +289,6 @@ class KeyMobileSitesSmoothPageSet(page_set_module.PageSet):
'http://www.sfgate.com/',
# Why: Non-latin character set
'http://worldjournal.com/',
- # Why: Mobile wiki
- 'http://www.wowwiki.com/World_of_Warcraft:_Mists_of_Pandaria',
# Why: #15 Alexa news
'http://online.wsj.com/home-page',
# Why: Image-heavy mobile site
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698