Don't always run the script tests if there are no compile targets.

scripts/slave/recipes/chromium_trybot_legacy.py

 # Copyright 2015 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.
 
 from infra.libs.infra_types import freeze
 
 DEPS = [
   'bot_update',
   'chromium',
   'chromium_android',
@@ skipping 421 matching lines @@
               'telemetry_unittests', 'telemetry_unittests.py',
               scripts_compile_targets),
           api.chromium.steps.ScriptTest(
               'telemetry_perf_unittests', 'telemetry_perf_unittests.py',
               scripts_compile_targets),
       ]
 
     # See if the patch needs to compile on the current platform.
     # Don't run analyze for other projects, such as blink, as there aren't that
     # many try jobs for them.
-    requires_compile = True
     if (isinstance(test_spec, dict) and
         api.properties.get('patch_project') == 'chromium'):
       analyze_config_file = bot_config['testing'].get('analyze_config_file',
                                          'trybot_analyze_config.json')
       requires_compile, matching_exes, compile_targets = \
           api.chromium_tests.analyze(
               get_test_names(gtest_tests) +
                   all_compile_targets(api, conditional_tests),
               compile_targets,
               analyze_config_file)
 
+      if not requires_compile:
+        return [], bot_update_step
+
       gtest_tests = filter_tests(gtest_tests, matching_exes)
 
     tests = []
 
     conditional_tests = tests_in_compile_targets(
         api, compile_targets, conditional_tests)
     tests.extend(find_test_named('telemetry_unittests', conditional_tests))
     tests.extend(find_test_named('telemetry_perf_unittests', conditional_tests))
     tests.extend(gtest_tests)
     tests.extend(find_test_named('nacl_integration', conditional_tests))
 
     if api.platform.is_win:
       tests.append(api.chromium.steps.MiniInstallerTest())
 
-    if not requires_compile:
-      # Even though the patch doesn't require compile, we'd still like to
-      # run tests not depending on compiled targets (that's obviously not
-      # covered by the "analyze" step).
-      tests = [t for t in tests if not t.compile_targets(api)]
-      return tests, bot_update_step
-
     has_swarming_tests = any(t.uses_swarming for t in tests)
 
     # Swarming uses Isolate to transfer files to swarming bots.
     # set_isolate_environment modifies GYP_DEFINES to enable test isolation.
     if bot_config.get('use_isolate') or has_swarming_tests:
       api.isolate.set_isolate_environment(api.chromium.c)
 
     # If going to use swarming_client (pinned in src/DEPS), ensure it is
     # compatible with what recipes expect.
     if has_swarming_tests:
@@ skipping 509 matching lines @@
             'test': 'base_tests',
             'args': ['--gtest-filter: *NaCl*.*'],
           },
         ],
       })
     ) +
     api.override_step_data(
       'analyze',
       api.json.output({'invalid_targets': ['invalid target', 'another one']}))
   )