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

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

Issue 2745103003: Add new trivial sites memory benchmark. (Closed)
Patch Set: rename benchmark Created 3 years, 9 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 | « testing/buildbot/chromium.perf.fyi.json ('k') | 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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 core import perf_benchmark 7 from core import perf_benchmark
8 8
9 from telemetry import benchmark 9 from telemetry import benchmark
10 from telemetry.timeline import chrome_trace_category_filter 10 from telemetry.timeline import chrome_trace_category_filter
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 def SetExtraBrowserOptions(self, options): 44 def SetExtraBrowserOptions(self, options):
45 # Just before we measure memory we flush the system caches 45 # Just before we measure memory we flush the system caches
46 # unfortunately this doesn't immediately take effect, instead 46 # unfortunately this doesn't immediately take effect, instead
47 # the next page run is effected. Due to this the first page run 47 # the next page run is effected. Due to this the first page run
48 # has anomalous results. This option causes us to flush caches 48 # has anomalous results. This option causes us to flush caches
49 # each time before Chrome starts so we effect even the first page 49 # each time before Chrome starts so we effect even the first page
50 # - avoiding the bug. 50 # - avoiding the bug.
51 options.clear_sytem_cache_for_browser_and_profile_on_start = True 51 options.clear_sytem_cache_for_browser_and_profile_on_start = True
52 52
53 53
54 @benchmark.Enabled('mac')
55 @benchmark.Enabled('win')
nednguyen 2017/03/14 01:03:21 Can you also add: @benchmark.Owner(emails=['erikch
56 class MemoryBenchmarkTrivialSitesDesktop(_MemoryInfra):
57 """Measure memory usage on trivial sites."""
58 page_set = page_sets.TrivialSitesStorySet
59 options = {'pageset_repeat': 5}
60
61 @classmethod
62 def Name(cls):
63 return 'memory.desktop'
64
65 @classmethod
66 def ShouldTearDownStateAfterEachStoryRun(cls):
67 return True
68
69 @classmethod
70 def ValueCanBeAddedPredicate(cls, value, is_first_result):
71 # TODO(crbug.com/610962): Remove this stopgap when the perf dashboard
72 # is able to cope with the data load generated by TBMv2 metrics.
73 return not _IGNORED_STATS_RE.search(value.name)
74
75
54 @benchmark.Enabled('android') # catapult:#3176 76 @benchmark.Enabled('android') # catapult:#3176
55 class MemoryBenchmarkTop10Mobile(_MemoryInfra): 77 class MemoryBenchmarkTop10Mobile(_MemoryInfra):
56 """Measure foreground/background memory on top 10 mobile page set. 78 """Measure foreground/background memory on top 10 mobile page set.
57 79
58 This metric provides memory measurements for the System Health Plan of 80 This metric provides memory measurements for the System Health Plan of
59 Chrome on Android. 81 Chrome on Android.
60 """ 82 """
61 page_set = page_sets.MemoryTop10Mobile 83 page_set = page_sets.MemoryTop10Mobile
62 options = {'pageset_repeat': 5} 84 options = {'pageset_repeat': 5}
63 85
(...skipping 175 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 of long running idle Gmail page """ 261 of long running idle Gmail page """
240 page_set = page_sets.LongRunningIdleGmailBackgroundPageSet 262 page_set = page_sets.LongRunningIdleGmailBackgroundPageSet
241 263
242 @classmethod 264 @classmethod
243 def Name(cls): 265 def Name(cls):
244 return 'memory.long_running_idle_gmail_background_tbmv2' 266 return 'memory.long_running_idle_gmail_background_tbmv2'
245 267
246 @classmethod 268 @classmethod
247 def ShouldDisable(cls, possible_browser): # http://crbug.com/616530 269 def ShouldDisable(cls, possible_browser): # http://crbug.com/616530
248 return cls.IsSvelte(possible_browser) 270 return cls.IsSvelte(possible_browser)
OLDNEW
« no previous file with comments | « testing/buildbot/chromium.perf.fyi.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698