[system-health] Re-enable non-verbose logging in System Health memory benchmarks

tools/perf/benchmarks/system_health.py

 # Copyright 2016 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 chrome_trace_category_filter
 from telemetry.web_perf import timeline_based_measurement
@@ skipping 46 matching lines @@
     return 'system_health.key_mobile_sites'
 
 
 class _MemorySystemHealthBenchmark(perf_benchmark.PerfBenchmark):
   """Chrome Memory System Health Benchmark.
 
   https://goo.gl/Jek2NL.
   """
 
   def SetExtraBrowserOptions(self, options):
+    # TODO(petrcermak): Remove this and switch to log-on-retry
+    # (http://crbug.com/623058).
+    options.logging_verbosity = options.NON_VERBOSE_LOGGING

[nednguyen, 2016/08/03 12:31:13]

Do we still crash failure that are not caught by cq tests? My hope is that we
don't need to do this to reduce extra-overhead on the browser.

[petrcermak, 2016/08/03 13:05:35]

Actually, apart from the disabled smoke tests, we only have 2 failures on
reference builds (desktop-reference + n5x-reference). Everythin else is enabled.
Are you suggesting that we only enable logging in the smoke tests?

[nednguyen, 2016/08/03 13:13:26]

Yes. I hope the smoke tests would defend for all crashes, hence hopefully
removes the need to enable logging in production mode.

[petrcermak, 2016/08/03 13:16:40]

Makes sense. In that case, let's abandon this patch.

     options.AppendExtraBrowserArgs([
         # TODO(perezju): Temporary workaround to disable periodic memory dumps.
         # See: http://crbug.com/513692
         '--enable-memory-benchmarking',
     ])
 
   def CreateTimelineBasedMeasurementOptions(self):
     options = timeline_based_measurement.Options(
         chrome_trace_category_filter.ChromeTraceCategoryFilter(
             '-*,disabled-by-default-memory-infra'))
@@ skipping 64 matching lines @@
     options.config.atrace_config.app_name = 'org.chromium.webview_shell'
     return options
 
   @classmethod
   def ShouldTearDownStateAfterEachStoryRun(cls):
     return True
 
   @classmethod
   def Name(cls):
     return 'system_health.webview_startup'