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

Unified Diff: tools/perf/benchmarks/memory_infra.py

Issue 1931533002: Port memory.long_running_idle_gmail benchmarks to TBMv2. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and remove deprecated method Created 4 years, 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/perf/benchmarks/memory.py ('k') | tools/perf/page_sets/long_running_idle_google_cases.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/benchmarks/memory_infra.py
diff --git a/tools/perf/benchmarks/memory_infra.py b/tools/perf/benchmarks/memory_infra.py
index 1b62c17daa2997a854446a190d99d91fc03548da..c152bf4a2b018cde33f99f5acd6d76d6227a7f82 100644
--- a/tools/perf/benchmarks/memory_infra.py
+++ b/tools/perf/benchmarks/memory_infra.py
@@ -10,7 +10,6 @@ from telemetry import benchmark
from telemetry.timeline import tracing_category_filter
from telemetry.web_perf import timeline_based_measurement
from telemetry.web_perf.metrics import memory_timeline
-from telemetry.web_perf.metrics import v8_gc_latency
import page_sets
@@ -144,37 +143,44 @@ class MemoryBenchmarkTop10Mobile(_MemoryInfra):
return 'memory.top_10_mobile'
+class _MemoryV8Benchmark(_MemoryInfra):
+ def CreateTimelineBasedMeasurementOptions(self):
+ v8_categories = [
+ 'blink.console', 'renderer.scheduler', 'v8', 'webkit.console']
+ memory_categories = ['blink.console', 'disabled-by-default-memory-infra']
+ category_filter = tracing_category_filter.TracingCategoryFilter(
+ ','.join(['-*'] + v8_categories + memory_categories))
+ options = timeline_based_measurement.Options(category_filter)
+ options.SetTimelineBasedMetric('v8AndMemoryMetrics')
+ return options
+
+
+
# Disabled on reference builds because they don't support the new
# Tracing.requestMemoryDump DevTools API.
# For 'reference' see http://crbug.com/540022.
# For 'android' see http://crbug.com/579546.
@benchmark.Disabled('reference', 'android')
-class MemoryLongRunningIdleGmailTBM(_MemoryInfra):
+class MemoryLongRunningIdleGmail(_MemoryV8Benchmark):
"""Use (recorded) real world web sites and measure memory consumption
of long running idle Gmail page """
page_set = page_sets.LongRunningIdleGmailPageSet
- def CreateTimelineBasedMeasurementOptions(self):
- v8_categories = [
- 'blink.console', 'renderer.scheduler', 'v8', 'webkit.console']
- memory_categories = 'blink.console,disabled-by-default-memory-infra'
- category_filter = tracing_category_filter.TracingCategoryFilter(
- memory_categories)
- for category in v8_categories:
- category_filter.AddIncludedCategory(category)
- options = timeline_based_measurement.Options(category_filter)
- return options
-
- def SetupBenchmarkDefaultTraceRerunOptions(self, tbm_options):
- tbm_options.SetLegacyTimelineBasedMetrics((
- v8_gc_latency.V8GCLatency(),
- memory_timeline.MemoryTimelineMetric(),
- ))
-
@classmethod
def Name(cls):
- return 'memory.long_running_idle_gmail_tbm'
+ return 'memory.long_running_idle_gmail_tbmv2'
+
+
+# Disabled on reference builds because they don't support the new
+# Tracing.requestMemoryDump DevTools API.
+# For 'reference' see http://crbug.com/540022.
+# For 'android' see http://crbug.com/579546.
+@benchmark.Disabled('reference', 'android')
+class MemoryLongRunningIdleGmailBackground(_MemoryV8Benchmark):
+ """Use (recorded) real world web sites and measure memory consumption
+ of long running idle Gmail page """
+ page_set = page_sets.LongRunningIdleGmailBackgroundPageSet
@classmethod
- def ShouldTearDownStateAfterEachStoryRun(cls):
- return True
+ def Name(cls):
+ return 'memory.long_running_idle_gmail_background_tbmv2'
« no previous file with comments | « tools/perf/benchmarks/memory.py ('k') | tools/perf/page_sets/long_running_idle_google_cases.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698