Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 # Copyright 2013 The Chromium Authors. All rights reserved. | 1 # Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
| 3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
| 4 | 4 |
| 5 from core import perf_benchmark | 5 from core import perf_benchmark |
| 6 | 6 |
| 7 from measurements import page_cycler | 7 from measurements import page_cycler |
| 8 import page_sets | 8 import page_sets |
| 9 from telemetry import benchmark | 9 from telemetry import benchmark |
| 10 | 10 |
| (...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 110 @classmethod | 110 @classmethod |
| 111 def ShouldDisable(cls, possible_browser): | 111 def ShouldDisable(cls, possible_browser): |
| 112 # http://crbug.com/597656 (Android Nexus 5X). | 112 # http://crbug.com/597656 (Android Nexus 5X). |
| 113 # http://crbug.com/605543 (Mac Snow Leopard). | 113 # http://crbug.com/605543 (Mac Snow Leopard). |
| 114 return (possible_browser.browser_type == 'reference' and ( | 114 return (possible_browser.browser_type == 'reference' and ( |
| 115 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X' or | 115 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X' or |
| 116 possible_browser.platform.GetOSVersionName() == 'snowleopard')) | 116 possible_browser.platform.GetOSVersionName() == 'snowleopard')) |
| 117 | 117 |
| 118 | 118 |
| 119 @benchmark.Enabled('android') | 119 @benchmark.Enabled('android') |
| 120 @benchmark.Disabled('all') # crbug.com/633222 | |
|
shatch
2016/08/02 16:02:14
Is this right? This is kinda weird, since enabled
nednguyen
2016/08/02 16:54:30
Yeah, Enabled is white-listing, whereas Disabled i
| |
| 120 class PageCyclerTop10Mobile(_PageCycler): | 121 class PageCyclerTop10Mobile(_PageCycler): |
| 121 """Page load time benchmark for the top 10 mobile web pages. | 122 """Page load time benchmark for the top 10 mobile web pages. |
| 122 | 123 |
| 123 Runs against pages recorded in November, 2013. | 124 Runs against pages recorded in November, 2013. |
| 124 """ | 125 """ |
| 125 | 126 |
| 126 @classmethod | 127 @classmethod |
| 127 def Name(cls): | 128 def Name(cls): |
| 128 return 'page_cycler.top_10_mobile' | 129 return 'page_cycler.top_10_mobile' |
| 129 | 130 |
| 130 def CreateStorySet(self, options): | 131 def CreateStorySet(self, options): |
| 131 # Disable the taobao.com page since it's crashing. crbug.com/509690 | 132 # Disable the taobao.com page since it's crashing. crbug.com/509690 |
| 132 stories = page_sets.Top10MobilePageSet(run_no_page_interactions=True) | 133 stories = page_sets.Top10MobilePageSet(run_no_page_interactions=True) |
| 133 found = next((x for x in stories if 'taobao.com' in x.url), None) | 134 found = next((x for x in stories if 'taobao.com' in x.url), None) |
| 134 if found: | 135 if found: |
| 135 stories.RemoveStory(found) | 136 stories.RemoveStory(found) |
| 136 return stories | 137 return stories |
| 137 | 138 |
| 138 @classmethod | |
| 139 def ShouldDisable(cls, possible_browser): | |
| 140 # http://crbug.com/633222 (Android Nexus 5). | |
| 141 return possible_browser.platform.GetDeviceTypeName() == 'Nexus 5' | |
| 142 | |
| 143 | 139 |
| 144 @benchmark.Disabled('android') # crbug.com/357326 | 140 @benchmark.Disabled('android') # crbug.com/357326 |
| 145 class PageCyclerToughLayoutCases(_PageCycler): | 141 class PageCyclerToughLayoutCases(_PageCycler): |
| 146 """Page loading for the slowest layouts observed in the Alexa top 1 million. | 142 """Page loading for the slowest layouts observed in the Alexa top 1 million. |
| 147 | 143 |
| 148 Recorded in July 2013. | 144 Recorded in July 2013. |
| 149 """ | 145 """ |
| 150 page_set = page_sets.ToughLayoutCasesPageSet | 146 page_set = page_sets.ToughLayoutCasesPageSet |
| 151 | 147 |
| 152 @classmethod | 148 @classmethod |
| (...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 198 set, without running in out-of-process iframes mode.. """ | 194 set, without running in out-of-process iframes mode.. """ |
| 199 page_set = page_sets.OopifBasicPageSet | 195 page_set = page_sets.OopifBasicPageSet |
| 200 | 196 |
| 201 @classmethod | 197 @classmethod |
| 202 def Name(cls): | 198 def Name(cls): |
| 203 return 'page_cycler.basic_oopif' | 199 return 'page_cycler.basic_oopif' |
| 204 | 200 |
| 205 @classmethod | 201 @classmethod |
| 206 def ShouldDisable(cls, possible_browser): | 202 def ShouldDisable(cls, possible_browser): |
| 207 return cls.IsSvelte(possible_browser) # http://crbug.com/607657 | 203 return cls.IsSvelte(possible_browser) # http://crbug.com/607657 |
| OLD | NEW |