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

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

Issue 2378053004: Disable page_cycler_v2 on N6 webview (Closed)
Patch Set: Created 4 years, 2 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 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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 """The page cycler v2. 5 """The page cycler v2.
6 6
7 For details, see design doc: 7 For details, see design doc:
8 https://docs.google.com/document/d/1EZQX-x3eEphXupiX-Hq7T4Afju5_sIdxPWYetj7ynd0 8 https://docs.google.com/document/d/1EZQX-x3eEphXupiX-Hq7T4Afju5_sIdxPWYetj7ynd0
9 """ 9 """
10 10
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 def ShouldDisable(cls, possible_browser): 48 def ShouldDisable(cls, possible_browser):
49 # crbug.com/619254 49 # crbug.com/619254
50 if possible_browser.browser_type == 'reference': 50 if possible_browser.browser_type == 'reference':
51 return True 51 return True
52 52
53 # crbug.com/616781 53 # crbug.com/616781
54 if (cls.IsSvelte(possible_browser) or 54 if (cls.IsSvelte(possible_browser) or
55 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X' or 55 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X' or
56 possible_browser.platform.GetDeviceTypeName() == 'AOSP on BullHead'): 56 possible_browser.platform.GetDeviceTypeName() == 'AOSP on BullHead'):
57 return True 57 return True
58
59 # crbug.com/651188
60 if (possible_browser.platform.GetDeviceTypeName() == 'Nexus 6' and
61 possible_browser.browser_type == 'android-webview'):
62 return True
63
58 return False 64 return False
59 65
60 66
61 class PageCyclerV2Typical25(_PageCyclerV2): 67 class PageCyclerV2Typical25(_PageCyclerV2):
62 """Page load time benchmark for a 25 typical web pages. 68 """Page load time benchmark for a 25 typical web pages.
63 69
64 Designed to represent typical, not highly optimized or highly popular web 70 Designed to represent typical, not highly optimized or highly popular web
65 sites. Runs against pages recorded in June, 2014. 71 sites. Runs against pages recorded in June, 2014.
66 """ 72 """
67 73
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 set, without running in out-of-process iframes mode.. """ 214 set, without running in out-of-process iframes mode.. """
209 page_set = page_sets.OopifBasicPageSet 215 page_set = page_sets.OopifBasicPageSet
210 216
211 @classmethod 217 @classmethod
212 def Name(cls): 218 def Name(cls):
213 return 'page_cycler_v2.basic_oopif' 219 return 'page_cycler_v2.basic_oopif'
214 220
215 def CreateStorySet(self, options): 221 def CreateStorySet(self, options):
216 return page_sets.OopifBasicPageSet(cache_temperatures=[ 222 return page_sets.OopifBasicPageSet(cache_temperatures=[
217 cache_temperature.PCV1_COLD, cache_temperature.PCV1_WARM]) 223 cache_temperature.PCV1_COLD, cache_temperature.PCV1_WARM])
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