Revert "[Android] Add context manager to record logcat during step(s)."

scripts/slave/recipe_modules/chromium_android/api.py

 # Copyright 2013 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 contextlib
 import datetime
 import json
 import os
 import re
 import urllib
@@ skipping 231 matching lines @@
     # (or at least make the difference clear).
     self.m.archive.zip_and_upload_build(
         'zip_build',
         target=self.m.chromium.c.BUILD_CONFIG,
         # We send None as the path so that zip_build.py gets it from factory
         # properties.
         build_url=None,
         src_dir=self.m.path['slave_build'].join('src'),
         exclude_files='lib.target,gen,android_webview,jingle_unittests')
 
+  def spawn_logcat_monitor(self):
+    self.m.step(
+        'spawn_logcat_monitor',
+        [self.m.path['build'].join('scripts', 'slave', 'daemonizer.py'),
+         '--', self.c.cr_build_android.join('adb_logcat_monitor.py'),
+         self.m.chromium.c.build_dir.join('logcat')],
+        env=self.m.chromium.get_env(),
+        infra_step=True)
+
   def spawn_device_monitor(self):
     script = self.m.path['build'].join('scripts', 'slave', 'daemonizer.py')
     args = [
         '--action', 'restart',
         '--pid-file-path', '/tmp/device_monitor.pid',
         '--', self.resource('spawn_device_monitor.py'),
         self.m.adb.adb_path(),
         json.dumps(self._devices),
         self.m.properties['mastername'],
         self.m.properties['buildername'],
@@ skipping 430 matching lines @@
         '--n', count,
         'shutdown',
     ]
     self.m.python(
         'shutdown_instances',
         self.resource('gce_manager.py'),
         args,
         infra_step=True,
     )
 
-  def spawn_logcat_monitor(self, logcat_dir=None):
-    if not logcat_dir:
-      logcat_dir = self.c.default_logcat_dir
-
-    self.m.step(
-        'spawn_logcat_monitor',
-        [self.m.path['build'].join('scripts', 'slave', 'daemonizer.py'),
-         '--', self.c.cr_build_android.join('adb_logcat_monitor.py'),
-         logcat_dir],
-        env=self.m.chromium.get_env(),
-        infra_step=True)
-
-  def logcat_dump(self, output_logcat_file, logcat_dir=None, gs_bucket=None):
-    if not logcat_dir:
-      logcat_dir = self.c.default_logcat_dir
-
+  def logcat_dump(self, gs_bucket=None):
     if gs_bucket:
+      log_path = self.m.chromium.output_dir.join('full_log')
       self.m.python(
           'logcat_dump',
           self.m.path['checkout'].join('build', 'android',
                                        'adb_logcat_printer.py'),
-          ['--output-path', output_logcat_file, logcat_dir],
+          [ '--output-path', log_path,
+            self.m.path['checkout'].join('out', 'logcat') ],
           infra_step=True)
-
       try:
         self.m.gsutil.upload(
-            output_logcat_file,
+            log_path,
             gs_bucket,
             'logcat_dumps/%s/%s' % (self.m.properties['buildername'],
                                     self.m.properties['buildnumber']),
             link_name='logcat dump',
             version='4.7',
             parallel_upload=True,
             timeout=300)
       except self.m.step.StepFailure:
         # TODO(bpastene): Remove the following if the blab gs issue is resolved
         # If the upload fails, change it to a warning and continue on with
         # the build. We don't want a failed logcat upload crashing a tryjob
         step_result = self.m.step.active_result
         step_result.presentation.status = self.m.step.WARNING
+
     else:
       self.m.python(
           'logcat_dump',
           self.m.path['build'].join('scripts', 'slave', 'tee.py'),
-          [output_logcat_file,
+          [self.m.chromium.output_dir.join('full_log'),
            '--',
            self.m.path['checkout'].join('build', 'android',
                                         'adb_logcat_printer.py'),
-           logcat_dir],
-          infra_step=True)
+           self.m.path['checkout'].join('out', 'logcat')],
+          infra_step=True,
+          )
 
-  @contextlib.contextmanager
-  def logcat(self, step_name):
-    """Context manager for recording logcat.
-
-    Args:
-      step_name: Name of the outer step. Any steps ran inside this context
-          manager will be nested inside.
-    """
-    with self.m.step.nest(step_name):
-      logcat_dir = self.m.path.mkdtemp('logcat')
-      self.spawn_logcat_monitor(logcat_dir)
-      yield
-      self.logcat_dump(logcat_dir=logcat_dir,
-                       output_logcat_file=self.m.raw_io.output('logcat'),
-                       gs_bucket=None)
-
-  def stack_tool_steps(self, logcat_file):
+  def stack_tool_steps(self):
+    log_file = self.m.path['checkout'].join('out',
+                                            self.m.chromium.c.BUILD_CONFIG,
+                                            'full_log')
     target_arch = self.m.chromium.c.gyp_env.GYP_DEFINES['target_arch']
     # gyp converts ia32 to x86, bot needs to do the same
     target_arch = {'ia32': 'x86'}.get(target_arch) or target_arch
     self.m.step(
         'stack_tool_with_logcat_dump',
         [self.m.path['checkout'].join('third_party', 'android_platform',
                               'development', 'scripts', 'stack'),
-         '--arch', target_arch, '--more-info', logcat_file],
+         '--arch', target_arch, '--more-info', log_file],
         env=self.m.chromium.get_env(),
         infra_step=True)
     self.m.step(
         'stack_tool_for_tombstones',
         [self.m.path['checkout'].join('build', 'android', 'tombstones.py'),
          '-a', '-s', '-w'], env=self.get_env(),
         infra_step=True)
     if self.c.asan_symbolize:
       self.m.step(
           'stack_tool_for_asan',
           [self.m.path['checkout'].join('build',
                                         'android',
                                         'asan_symbolize.py'),
-           '-l', logcat_file], env=self.m.chromium.get_env(),
+           '-l', log_file], env=self.m.chromium.get_env(),
           infra_step=True)
 
   def test_report(self):
     self.m.python.inline(
         'test_report',
          """
             import glob, os, sys
             for report in glob.glob(sys.argv[1]):
               with open(report, 'r') as f:
                 for l in f.readlines():
@@ skipping 25 matching lines @@
         kwargs = {}
       self.provision_devices(**kwargs)
       if self.m.chromium.c.gyp_env.GYP_DEFINES.get('asan', 0) == 1:
         self.asan_device_setup()
 
       self.spawn_device_monitor()
 
   def common_tests_final_steps(self, logcat_gs_bucket=None):
     if not self.c.gce_setup:
       self.shutdown_device_monitor()
-    logcat_file = self.m.chromium.output_dir.join('full_log')
-    self.logcat_dump(output_logcat_file=logcat_file,
-                     gs_bucket=logcat_gs_bucket)
-    self.stack_tool_steps(logcat_file=logcat_file)
+    self.logcat_dump(gs_bucket=logcat_gs_bucket)
+    self.stack_tool_steps()
     if self.c.gce_setup:
       self.shutdown_gce_instances()
     self.test_report()
 
   def run_bisect_script(self, extra_src='', path_to_config='', **kwargs):
     self.m.step('prepare bisect perf regression',
         [self.m.path['checkout'].join('tools',
                                       'prepare-bisect-perf-regression.py'),
          '-w', self.m.path['slave_build']])
 
@@ skipping 258 matching lines @@
   def test_runner(self, step_name, args=None, **kwargs):
     """Wrapper for the python testrunner script.
 
     Args:
       step_name: Name of the step.
       args: Testrunner arguments.
     """
     with self.handle_exit_codes():
       return self.m.python(
           step_name, self.c.test_runner, args, **kwargs)