[tools/perf] Only send _avg metrics for memory infra metrics

tools/perf/benchmarks/memory_infra.py

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..7bc3053ffb252772b2a033ea2db9132c2bf79b2b 100644
--- a/tools/perf/benchmarks/memory_infra.py
+++ b/tools/perf/benchmarks/memory_infra.py
@@ -89,6 +89,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 value.name.endswith('_avg')

[petrcermak, 2016/05/24 10:23:52]

Ned: Will this method also be applied to things like num_failed? If so, then we
should do the following:

  return 'memory:' not in value.name or value.name.endswith('_avg')

(I don't think we can use value.name.startswith('memory:') because the tir_label
might have already been added to the front of the value name).

Furthermore, this code will remove all dump counts (because they're simple
scalars), so I think that we should do the following instead:

  _IGNORED_STATS_RE = re.compile(r'_(std|count|max|min|sum|pct_\d{4}(_\d+)?)$')

  return not _IGNORED_STATS_RE.search(value.name)

See
https://github.com/catapult-project/catapult/blob/master/tracing/tracing/valu....

[perezju, 2016/05/24 11:29:06]

You're correct, this was throwing away dump counts. I took your suggestion, and
did a few other manual checks to ensure doesn't throw away too much or too
little.

+
 
 # Benchmark is disabled by default because it takes too long to run.
 @benchmark.Disabled('all')
@@ -107,6 +113,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 value.name.endswith('_avg')
+
 
 # TODO(bashi): Workaround for http://crbug.com/532075
 # @benchmark.Enabled('android') shouldn't be needed.