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

Side by Side Diff: tools/perf/benchmarks/page_cycler.py

Issue 2214773002: [perf] Disable page_cycler.typical_25.reference on Mac Yosemite (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 131
132 class PageCyclerTypical25(_PageCycler): 132 class PageCyclerTypical25(_PageCycler):
133 """Page load time benchmark for a 25 typical web pages. 133 """Page load time benchmark for a 25 typical web pages.
134 134
135 Designed to represent typical, not highly optimized or highly popular web 135 Designed to represent typical, not highly optimized or highly popular web
136 sites. Runs against pages recorded in June, 2014. 136 sites. Runs against pages recorded in June, 2014.
137 """ 137 """
138 options = {'pageset_repeat': 3} 138 options = {'pageset_repeat': 3}
139 139
140 @classmethod 140 @classmethod
141 def ShouldDisable(cls, possible_browser): # http://crbug.com/597656 141 def ShouldDisable(cls, possible_browser):
142 if (possible_browser.browser_type == 'reference' and 142 if possible_browser.browser_type == 'reference':
143 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X'): 143 if possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X':
144 return True 144 return True # http://crbug.com/597656
145 # http://crbug.com/616781 145 if possible_browser.platform.GetOSVersionName() == 'yosemite':
146 return True # http://crbug.com/634337
146 if possible_browser.platform.GetDeviceTypeName() == 'AOSP on BullHead': 147 if possible_browser.platform.GetDeviceTypeName() == 'AOSP on BullHead':
147 return True 148 return True # http://crbug.com/616781
148 return False 149 return False
149 150
150 @classmethod 151 @classmethod
151 def Name(cls): 152 def Name(cls):
152 return 'page_cycler.typical_25' 153 return 'page_cycler.typical_25'
153 154
154 def CreateStorySet(self, options): 155 def CreateStorySet(self, options):
155 return page_sets.Typical25PageSet(run_no_page_interactions=True) 156 return page_sets.Typical25PageSet(run_no_page_interactions=True)
156 157
157 158
(...skipping 15 matching lines...) Expand all
173 set, without running in out-of-process iframes mode.. """ 174 set, without running in out-of-process iframes mode.. """
174 page_set = page_sets.OopifBasicPageSet 175 page_set = page_sets.OopifBasicPageSet
175 176
176 @classmethod 177 @classmethod
177 def Name(cls): 178 def Name(cls):
178 return 'page_cycler.basic_oopif' 179 return 'page_cycler.basic_oopif'
179 180
180 @classmethod 181 @classmethod
181 def ShouldDisable(cls, possible_browser): 182 def ShouldDisable(cls, possible_browser):
182 return cls.IsSvelte(possible_browser) # http://crbug.com/607657 183 return cls.IsSvelte(possible_browser) # http://crbug.com/607657
OLDNEW
« 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