| Index: tools/perf/benchmarks/memory_infra.py
|
| diff --git a/tools/perf/benchmarks/memory_infra.py b/tools/perf/benchmarks/memory_infra.py
|
| index 4672f73bf25dc4e5c0547c6afd10ba549efe71e7..7904eaeace8d6b472263c972176f183735925f16 100644
|
| --- a/tools/perf/benchmarks/memory_infra.py
|
| +++ b/tools/perf/benchmarks/memory_infra.py
|
| @@ -14,6 +14,11 @@ from telemetry.web_perf.metrics import memory_timeline
|
| import page_sets
|
|
|
|
|
| +# See tr.v.Numeric.getSummarizedScalarNumericsWithNames()
|
| +# https://github.com/catapult-project/catapult/blob/master/tracing/tracing/value/numeric.html#L323
|
| +_IGNORED_STATS_RE = re.compile(r'_(std|count|max|min|sum|pct_\d{4}(_\d+)?)$')
|
| +
|
| +
|
| class _MemoryInfra(perf_benchmark.PerfBenchmark):
|
| """Base class for new-generation memory benchmarks based on memory-infra.
|
|
|
| @@ -89,6 +94,12 @@ class TBMv2MemoryBenchmarkTop10Mobile(MemoryHealthPlan):
|
| def Name(cls):
|
| return 'memory.top_10_mobile_tbmv2'
|
|
|
| + @classmethod
|
| + def ValueCanBeAddedPredicate(cls, value, is_first_result):
|
| + # TODO(crbug.com/610962): Remove this stopgap when the perf dashboard
|
| + # is able to cope with the data load generated by TBMv2 metrics.
|
| + return not _IGNORED_STATS_RE.search(value.name)
|
| +
|
|
|
| # Benchmark is disabled by default because it takes too long to run.
|
| @benchmark.Disabled('all')
|
| @@ -107,6 +118,12 @@ class DualBrowserBenchmark(_MemoryInfra):
|
| def Name(cls):
|
| return 'memory.dual_browser_test'
|
|
|
| + @classmethod
|
| + def ValueCanBeAddedPredicate(cls, value, is_first_result):
|
| + # TODO(crbug.com/610962): Remove this stopgap when the perf dashboard
|
| + # is able to cope with the data load generated by TBMv2 metrics.
|
| + return not _IGNORED_STATS_RE.search(value.name)
|
| +
|
|
|
| # TODO(bashi): Workaround for http://crbug.com/532075
|
| # @benchmark.Enabled('android') shouldn't be needed.
|
|
|