Simplified android_builder and moved internal-only methods to internal module

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 os
 
 from slave import recipe_api
 
 class AndroidApi(recipe_api.RecipeApi):
   def __init__(self, **kwargs):
@@ skipping 53 matching lines @@
     Args:
       step_name: Name of the step.
       archive_name: Name of the archive file.
       files: List of files. Files can be glob's or file paths. If no files
         are provided, everything in the target directory will be included.
       preserve_paths: If True, files will be stored using the subdolders
         in the archive.
     """
     archive_args = ['--target', self.m.chromium.c.BUILD_CONFIG,
                     '--name', archive_name]
-    if files:
+
+    # These are covered by build_internal.  Bleh. -luqui
+    if files:              # pragma: no cover
       archive_args.extend(['--files', ','.join(files)])
-    if not preserve_paths:
+    if not preserve_paths: # pragma: no cover
       archive_args.append('--ignore-subfolder-names')
 
     yield self.m.python(
       step_name,
       str(self.m.path['build'].join(
           'scripts', 'slave', 'android', 'archive_build.py')),
       archive_args,
       always_run=True,
       **kwargs
     )
@@ skipping 85 matching lines @@
     repos = ['src', 'src-internal']
     if self.c.REPO_NAME not in repos:
       repos.append(self.c.REPO_NAME)
     # TODO(sivachandra): Disable subannottations after cleaning up
     # tree_truth.sh.
     yield self.m.step('tree truth steps',
                       [self.m.path['checkout'].join('build', 'tree_truth.sh'),
                        self.m.path['checkout']] + repos,
                       allow_subannotations=False)
 
-  def runhooks(self, extra_env=None):
-    run_hooks_env = self.get_env()
-    if self.c.INTERNAL:
-      run_hooks_env['EXTRA_LANDMINES_SCRIPT'] = self.internal_dir.join(
-        'build', 'get_internal_landmines.py')
-    if extra_env:
-      run_hooks_env.update(extra_env)
-    return self.m.chromium.runhooks(env=run_hooks_env)
+  def runhooks(self, extra_env={}):
+    return self.m.chromium.runhooks(env=dict(self.get_env().items() +
+                                             extra_env.items()))
 
   def apply_svn_patch(self):
     # TODO(sivachandra): We should probably pull this into its own module
     # (maybe a 'tryserver' module) at some point.
     return self.m.step(
         'apply_patch',
         [self.m.path['build'].join('scripts', 'slave', 'apply_svn_patch.py'),
          '-p', self.m.properties['patch_url'],
          '-r', self.internal_dir])
 
   def compile(self, **kwargs):
     assert 'env' not in kwargs, (
         "chromium_andoid compile clobbers env in keyword arguments")
     kwargs['env'] = self.get_env()
     return self.m.chromium.compile(**kwargs)
 
-  def findbugs(self):
-    assert self.c.INTERNAL, 'findbugs is only available on internal builds'
-    cmd = [
-        self.m.path['checkout'].join('build', 'android', 'findbugs_diff.py'),
-        '-b', self.internal_dir.join('bin', 'findbugs_filter'),
-        '-o', 'com.google.android.apps.chrome.-,org.chromium.-',
-    ]
-    yield self.m.step('findbugs internal', cmd, env=self.get_env())
-
-    # If findbugs fails, there could be stale class files. Delete them, and
-    # next run maybe we'll do better.
-    if self.m.step_history.last_step().retcode != 0:
-      yield self.m.path.rmwildcard(
-          '*.class',
-          self.m.path['checkout'].join('out'),
-          always_run=True)
-
-  def checkdeps(self):
-    assert self.c.INTERNAL, 'checkdeps is only available on internal builds'
-    yield self.m.step(
-      'checkdeps',
-      [self.m.path['checkout'].join('tools', 'checkdeps', 'checkdeps.py'),
-       '--root=%s' % self.internal_dir],
-      env=self.get_env())
-
-  def lint(self):
-    assert self.c.INTERNAL, 'lint is only available on internal builds'
-    yield self.m.step(
-        'lint',
-        [self.internal_dir.join('bin', 'lint.py')],
-        env=self.get_env())
-
   def git_number(self):
     yield self.m.step(
         'git_number',
         [self.m.path['depot_tools'].join('git_number.py')],
         stdout = self.m.raw_io.output(),
         step_test_data=(
           lambda:
             self.m.raw_io.test_api.stream_output('3000\n')
         ),
         cwd=self.m.path['checkout'])
 
-  def _upload_build(self, bucket, path):
+  def upload_build(self, bucket, path):
     archive_name = 'build_product.zip'
 
     zipfile = self.m.path['checkout'].join('out', archive_name)
     self._cleanup_list.append(zipfile)
 
     yield self.make_zip_archive(
       'zip_build_product',
       archive_name,
       preserve_paths=True,
       cwd=self.m.path['checkout']
     )
 
     yield self.m.gsutil.upload(
         name='upload_build_product',
         source=zipfile,
         bucket=bucket,
         dest=path
     )
 
-  def upload_clusterfuzz(self):
-    revision = self.m.properties['revision']
-    # When unpacking, ".." will be stripped from the path and the library will
-    # end up in ./third_party/llvm-build/...
-    files = ['apks/*', 'lib/*.so',
-             '../../third_party/llvm-build/Release+Asserts/lib/clang/*/lib/' +
-             'linux/libclang_rt.asan-arm-android.so']
-
-    archive_name = 'clusterfuzz.zip'
-    zipfile = self.m.path['checkout'].join('out', archive_name)
-    self._cleanup_list.append(zipfile)
-
-    yield self.git_number()
-    git_number = str.strip(self.m.step_history['git_number'].stdout)
-
-    yield self.make_zip_archive(
-      'zip_clusterfuzz',
-      archive_name,
-      files=files,
-      preserve_paths=False,
-      cwd=self.m.path['checkout']
-    )
-    yield self.m.python(
-        'git_revisions',
-        self.m.path['checkout'].join(self.c.internal_dir_name, 'build',
-                                     'clusterfuzz_generate_revision.py'),
-        ['--file', git_number],
-        always_run=True,
-    )
-    yield self.m.gsutil.upload(
-        name='upload_revision_data',
-        source=self.m.path['checkout'].join('out', git_number),
-        bucket='%s/revisions' % self.c.storage_bucket,
-        dest=git_number
-    )
-    yield self.m.gsutil.upload(
-        name='upload_clusterfuzz',
-        source=zipfile,
-        bucket=self.c.storage_bucket,
-        dest='%s%s.zip' % (self.c.upload_dest_prefix, git_number)
-    )
-
   def upload_build_for_tester(self):
-    return self._upload_build(
+    return self.upload_build(
         bucket=self._internal_names['BUILD_BUCKET'],
         path='%s/build_product_%s.zip' % (
             self.m.properties['buildername'], self.m.properties['revision']))
 
   def _download_build(self, bucket, path):
     base_path = path.split('/')[-1]
     zipfile = self.m.path['checkout'].join('out', base_path)
     self._cleanup_list.append(zipfile)
     yield self.m.gsutil.download(
         name='download_build_product',
@@ skipping 291 matching lines @@
 
     yield self.m.gsutil.upload(
         source=self.coverage_dir.join('coverage_html'),
         bucket='chrome-code-coverage',
         dest=gs_dest,
         args=['-R'],
         name='upload coverage report',
         link_name='Coverage report',
         always_run=True,
         **kwargs)