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

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

Issue 2611753005: [Android] Disable system_health.common_mobile on nexus 7 devices. (Closed)
Patch Set: Created 3 years, 11 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 import re 5 import re
6 6
7 from benchmarks import page_cycler_v2 7 from benchmarks import page_cycler_v2
8 8
9 from core import perf_benchmark 9 from core import perf_benchmark
10 10
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 """Mobile Chrome Energy System Health Benchmark.""" 70 """Mobile Chrome Energy System Health Benchmark."""
71 PLATFORM = 'mobile' 71 PLATFORM = 'mobile'
72 72
73 @classmethod 73 @classmethod
74 def ShouldDisable(cls, possible_browser): 74 def ShouldDisable(cls, possible_browser):
75 # http://crbug.com/612144 75 # http://crbug.com/612144
76 if (possible_browser.browser_type == 'reference' and 76 if (possible_browser.browser_type == 'reference' and
77 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X'): 77 possible_browser.platform.GetDeviceTypeName() == 'Nexus 5X'):
78 return True 78 return True
79 79
80 # http://crbug.com/667470
81 if 'Nexus 7' in possible_browser.platform.GetDeviceTypeName():
82 return True
83
80 return possible_browser.platform.GetDeviceTypeName() == 'Desktop' 84 return possible_browser.platform.GetDeviceTypeName() == 'Desktop'
81 85
82 86
83 class _MemorySystemHealthBenchmark(perf_benchmark.PerfBenchmark): 87 class _MemorySystemHealthBenchmark(perf_benchmark.PerfBenchmark):
84 """Chrome Memory System Health Benchmark. 88 """Chrome Memory System Health Benchmark.
85 89
86 This test suite is run separately from the common one due to the high overhead 90 This test suite is run separately from the common one due to the high overhead
87 of memory tracing. 91 of memory tracing.
88 92
89 https://goo.gl/Jek2NL. 93 https://goo.gl/Jek2NL.
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
185 and load a blank page with multiprocess enabled. 189 and load a blank page with multiprocess enabled.
186 """ 190 """
187 191
188 def SetExtraBrowserOptions(self, options): 192 def SetExtraBrowserOptions(self, options):
189 options.AppendExtraBrowserArgs( 193 options.AppendExtraBrowserArgs(
190 ['--webview-sandboxed-renderer']) 194 ['--webview-sandboxed-renderer'])
191 195
192 @classmethod 196 @classmethod
193 def Name(cls): 197 def Name(cls):
194 return 'system_health.webview_startup_multiprocess' 198 return 'system_health.webview_startup_multiprocess'
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