Make clusterfuzz_archive include llvm-symbolizer and sancov.

scripts/slave/recipe_modules/archive/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 re
 import sys
 
 import manual_bisect_files
 from recipe_engine import recipe_api
 
@@ skipping 252 matching lines @@
     cp_branch, cp_number = self.m.commit_position.parse(commit_position)
     build_git_commit = self._get_git_commit(update_properties, primary_project)
 
     if fixed_staging_dir:
       staging_dir = self.m.path['slave_build'].join('chrome_staging')
       self.m.file.rmtree('purge staging dir', staging_dir)
       self.m.file.makedirs('create staging dir', staging_dir)
     else:
       staging_dir = self.m.path.mkdtemp('chrome_staging')
 
+    llvm_tools_to_copy = ['llvm-symbolizer', 'sancov']
+    llvm_bin_dir = self.m.path['checkout'].join('third_party', 'llvm-build',
+                                                'Release+Asserts', 'bin')
+    ext = ''

[Nico, 2016/10/06 18:54:47]

nit: This can be

  ext = '.exe' if self.m.platform.name == 'win' else ''

if you want. Also, I think there's self.m.platform.is_win which is a bit
shorter.

[Oliver Chang, 2016/10/06 19:01:32]

Done.

+    if self.m.platform.name == 'win':
+      ext = '.exe'
+
+    for tool in llvm_tools_to_copy:
+      self.m.file.copy('Copy ' + tool,
+                       self.m.path.join(llvm_bin_dir, tool + ext), build_dir)
+
     # Build the list of files to archive.
     zip_file_list = [f for f in self.m.file.listdir('build_dir', build_dir)
                      if self._cf_should_package_file(f)]
 
     # Use the legacy platform name as Clusterfuzz has some expectations on
     # this (it only affects Windows, where it replace 'win' by 'win32').
     pieces = [self.legacy_platform_name(), target.lower()]
     if archive_subdir_suffix:
       pieces.append(archive_subdir_suffix)
     subdir = '-'.join(pieces)
@@ skipping 153 matching lines @@
   def legacy_download_url(self, gs_bucket_name, extra_url_components=None):
     """Returns a url suitable for downloading a Chromium build from
     Google Storage.
 
     extra_url_components, if specified, should be a string without a
     trailing '/' which is inserted in the middle of the URL.
 
     The builder_name, or parent_buildername, is always automatically
     inserted into the URL."""
     return self._legacy_url(True, gs_bucket_name, extra_url_components)