Use simpler "analyze" logic for chromium_trybot recipe on non-Windows

scripts/slave/recipe_modules/chromium_tests/api.py

 # Copyright 2014 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 copy
 import json
 
 from infra.libs.infra_types import freeze, thaw
 from recipe_engine import recipe_api
@@ skipping 490 matching lines @@
           raise
         if has_failing_swarming_tests:
           self.m.isolate.isolate_tests(self.m.chromium.output_dir,
                                        verbose=True)
 
     with self.wrap_chromium_tests(mastername):
       return self.m.test_utils.determine_new_failures(api, tests,
                                                       deapply_patch_fn)
 
   def analyze(self, affected_files, exes, compile_targets, config_file_name,
-              additional_names=None):
+              additional_names=None, legacy_postprocess=True):
     """Runs "analyze" step to determine targets affected by the patch.
 
     Returns a tuple of:
       - boolean, indicating whether patch requires compile
       - list of matching exes (see filter recipe module)
       - list of targets that need to be compiled (see filter recipe module)"""
 
     original_exes = exes[:]
     original_compile_targets = compile_targets[:]
 
     if additional_names is None:
       additional_names = ['chromium']
 
     use_mb = (self.m.chromium.c.project_generator.tool == 'mb')
     build_output_dir = '//out/%s' % self.m.chromium.c.build_config_fs
     self.m.filter.does_patch_require_compile(
         affected_files,
         exes=exes,
         compile_targets=compile_targets,
         additional_names=additional_names,
         config_file_name=config_file_name,
         use_mb=use_mb,
         build_output_dir=build_output_dir,
         cros_board=self.m.chromium.c.TARGET_CROS_BOARD)
 
     if not self.m.filter.result:
       # Patch does not require compile.
       return False, [], []
 
-    if 'all' in compile_targets:
+    if legacy_postprocess:
+      if 'all' in compile_targets:
+        compile_targets = self.m.filter.compile_targets
+      else:
+        compile_targets = list(set(compile_targets) &
+                               set(self.m.filter.compile_targets))
+      # Always add |matching_exes|. They will be covered by |compile_targets|,
+      # but adding |matching_exes| makes determing if conditional tests are
+      # necessary easier. For example, if we didn't do this we could end up
+      # with chrome_run as a compile_target and not chrome (since chrome_run
+      # depends upon chrome). This results in not picking up
+      # NaclIntegrationTest as it depends upon chrome not chrome_run.
+      compile_targets = list(set(self.m.filter.matching_exes + compile_targets))
+    else:
       compile_targets = self.m.filter.compile_targets
-    else:
-      compile_targets = list(set(compile_targets) &
-                             set(self.m.filter.compile_targets))
-    # Always add |matching_exes|. They will be covered by |compile_targets|,
-    # but adding |matching_exes| makes determing if conditional tests are
-    # necessary easier. For example, if we didn't do this we could end up
-    # with chrome_run as a compile_target and not chrome (since chrome_run
-    # depends upon chrome). This results in not picking up
-    # NaclIntegrationTest as it depends upon chrome not chrome_run.
-    compile_targets = list(set(self.m.filter.matching_exes + compile_targets))
 
     # Add crash_service to compile_targets. This is done after filtering compile
     # targets out because crash_service should always be there on windows.
     # TODO(akuegel): Need to solve this in a better way. crbug.com/478053
     if (self.m.platform.is_win and compile_targets and
         'crash_service' not in compile_targets and not use_mb):
       compile_targets.extend(['crash_service'])
 
     # Emit more detailed output useful for debugging.
     analyze_details = {
@@ skipping 115 matching lines @@
   def get_compile_targets_for_scripts(self):
     return self.m.python(
         name='get compile targets for scripts',
         script=self.m.path['checkout'].join(
             'testing', 'scripts', 'get_compile_targets.py'),
         args=[
             '--output', self.m.json.output(),
             '--',
         ] + self.get_common_args_for_scripts(),
         step_test_data=lambda: self.m.json.test_api.output({}))