Memory Infra: Replace memory_health_plan with top_10_mobile

tools/perf/benchmarks/memory_infra.py

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import re
 
 from core import perf_benchmark
 
 from telemetry import benchmark
 from telemetry.timeline import tracing_category_filter
@@ skipping 39 matching lines @@
       # in third_party/catapult).
       tbm_options.SetTimelineBasedMetric('memoryMetric')
     else:
       raise Exception('Unrecognized TBM version: %s' % self.TBM_VERSION)
     return tbm_options
 
 
 # TODO(bashi): Workaround for http://crbug.com/532075
 # @benchmark.Enabled('android') shouldn't be needed.
 @benchmark.Enabled('android')
-class MemoryHealthQuick(_MemoryInfra):
-  """Timeline based benchmark for the Memory Health Plan (1 iteration)."""
-  page_set = page_sets.MemoryHealthStory
+class MemoryBenchmarkTop10Mobile(_MemoryInfra):
+  """Measure foreground/background memory on top 10 mobile page set.
+
+  This metric provides memory measurements for the System Health Plan of
+  Chrome on Android.
+  """
+  page_set = page_sets.MemoryTop10Mobile
+  options = {'pageset_repeat': 5}
 
   @classmethod
   def Name(cls):
-    return 'memory.memory_health_quick'
+    return 'memory.top_10_mobile'
 
   @classmethod
   def ShouldDisable(cls, possible_browser):
     # TODO(crbug.com/586148): Benchmark should not depend on DeskClock app.
     return not possible_browser.platform.CanLaunchApplication(
         'com.google.android.deskclock')
 
 
-# Benchmark is disabled by default because it takes too long to run.
-@benchmark.Disabled('all')
-class MemoryHealthPlan(MemoryHealthQuick):
-  """Timeline based benchmark for the Memory Health Plan (5 iterations)."""
-  options = {'pageset_repeat': 5}
-
-  @classmethod
-  def Name(cls):
-    return 'memory.memory_health_plan'
-
-
+# TODO(bashi): Workaround for http://crbug.com/532075
+# @benchmark.Enabled('android') shouldn't be needed.
 @benchmark.Enabled('android')
-class TBMv2MemoryBenchmarkTop10Mobile(MemoryHealthQuick):
-  """Timeline based benchmark for the Memory Health Plan based on TBMv2.
+class TBMv2MemoryBenchmarkTop10Mobile(MemoryBenchmarkTop10Mobile):
+  """Measure foreground/background memory on top 10 mobile page set (TBMv2).
 
   This is a temporary benchmark to compare the new TBMv2 memory metric
   (memory_metric.html) with the existing TBMv1 one (memory_timeline.py). Once
   all issues associated with the TBMv2 metric are resolved, all memory
   benchmarks (including the ones in this file) will switch to use it instead
   of the TBMv1 metric and this temporary benchmark will be removed. See
   crbug.com/60361.
   """
   TBM_VERSION = 2
 
@@ skipping 24 matching lines @@
 
   @classmethod
   def Name(cls):
     return 'memory.blink_memory_mobile'
 
   @classmethod
   def ValueCanBeAddedPredicate(cls, value, is_first_result):
     return bool(cls._RE_RENDERER_VALUES.match(value.name))
 
 
-# Disabled on reference builds because they don't support the new

[perezju, 2016/04/29 12:58:06]

Ignore this bit, this should happen at
https://codereview.chromium.org/1936493002/

-# Tracing.requestMemoryDump DevTools API. See http://crbug.com/540022.
-@benchmark.Disabled('reference')
-class MemoryBenchmarkTop10Mobile(_MemoryInfra):
-  """Timeline based benchmark for measuring memory on top 10 mobile sites."""
-
-  page_set = page_sets.MemoryInfraTop10MobilePageSet
-
-  @classmethod
-  def Name(cls):
-    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 MemoryLongRunningIdleGmail(_MemoryV8Benchmark):
   """Use (recorded) real world web sites and measure memory consumption
   of long running idle Gmail page """
   page_set = page_sets.LongRunningIdleGmailPageSet
 
   @classmethod
   def Name(cls):
     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 Name(cls):
     return 'memory.long_running_idle_gmail_background_tbmv2'