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

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

Issue 2130683002: [Experiment] purge-and-suspend Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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
Index: tools/perf/benchmarks/memory_infra.py
diff --git a/tools/perf/benchmarks/memory_infra.py b/tools/perf/benchmarks/memory_infra.py
index c72a7083f94f6cdf730081d89dda8a9a7c277ab0..6eaccc2bf795347145a555e461e1eb890a00b5b4 100644
--- a/tools/perf/benchmarks/memory_infra.py
+++ b/tools/perf/benchmarks/memory_infra.py
@@ -31,6 +31,8 @@ class _MemoryInfra(perf_benchmark.PerfBenchmark):
# TODO(perezju): Temporary workaround to disable periodic memory dumps.
# See: http://crbug.com/513692
'--enable-memory-benchmarking',
+ # TODO(ssid): Remove this flag after fixing http://crbug.com/461788.
+ '--no-sandbox',
])
def CreateTimelineBasedMeasurementOptions(self):
@@ -223,3 +225,34 @@ class MemoryLongRunningIdleGmailBackground(_MemoryV8Benchmark):
@classmethod
def ShouldDisable(cls, possible_browser): # http://crbug.com/616530
return cls.IsSvelte(possible_browser)
+
+
+@benchmark.Enabled('android')
+class RendererMemoryTest(_MemoryInfra):
+ """Timeline based benchmark for measuring memory consumption on mobile
+ sites on which blink's memory consumption is relatively high."""
+
+ TBM_VERSION = 2
+
+ page_set = page_sets.MemoryTestPageSet
+
+ def SetExtraBrowserOptions(self, options):
+ super(RendererMemoryTest, self).SetExtraBrowserOptions(
+ options)
+ options.AppendExtraBrowserArgs([
+ # Ignore certs errors because record_wpr cannot handle certs correctly
+ # in some cases (e.g. WordPress).
+ '--ignore-certificate-errors',
+ # Add purge-and-suspend-time.
+ '--purge-and-suspend-time=10',
+ '--enable-heap-profiling',
+ ])
+
+ @classmethod
+ def Name(cls):
+ return 'memory.memory_test'
+
+ @classmethod
+ def ValueCanBeAddedPredicate(cls, value, is_first_result):
+ return (not _IGNORED_STATS_RE.search(value.name) and
+ 'renderer_processes' in value.name)

Powered by Google App Engine
This is Rietveld 408576698