Chromium Code Reviews| 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 |