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

scripts/slave/recipes/chromium_trybot.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.
 
 from infra.libs.infra_types import freeze
 
 DEPS = [
   'bot_update',
   'chromium',
   'chromium_android',
@@ skipping 526 matching lines @@
           master_dict,
           override_bot_type='builder_tester',
           override_tests=tests)
 
   requires_compile, _, compile_targets = \
       api.chromium_tests.analyze(
           all_compile_targets(api, tests + tests_including_triggered),
           compile_targets,
           'trybot_analyze_config.json')
 
-  if requires_compile:
-    tests = tests_in_compile_targets(api, compile_targets, tests)
-    tests_including_triggered = tests_in_compile_targets(
-        api, compile_targets, tests_including_triggered)
+  if not requires_compile:
+    return
 
-    api.chromium_tests.compile_specific_targets(
-        main_waterfall_config['mastername'],
-        main_waterfall_config['buildername'],
-        bot_update_step,
-        master_dict,
-        test_spec,
-        compile_targets,
-        tests_including_triggered,
-        override_bot_type='builder_tester',
-        disable_isolate=bot_config.get('disable_isolate', False))
-  else:
-    # 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)]
+  tests = tests_in_compile_targets(api, compile_targets, tests)
+  tests_including_triggered = tests_in_compile_targets(
+      api, compile_targets, tests_including_triggered)
+
+  api.chromium_tests.compile_specific_targets(
+      main_waterfall_config['mastername'],
+      main_waterfall_config['buildername'],
+      bot_update_step,
+      master_dict,
+      test_spec,
+      compile_targets,
+      tests_including_triggered,
+      override_bot_type='builder_tester',
+      disable_isolate=bot_config.get('disable_isolate', False))
 
   def deapply_patch_fn(failing_tests):
     api.chromium_tests.deapply_patch(bot_update_step)
     compile_targets = list(api.itertools.chain(
         *[t.compile_targets(api) for t in failing_tests]))
     if compile_targets:
       # Remove duplicate targets.
       compile_targets = sorted(set(compile_targets))
       # Search for *.isolated only if enabled in bot config or if some
       # swarming test is being recompiled.
@@ skipping 480 matching lines @@
                      tryserver=True)
   yield step_failure(mastername='tryserver.chromium.linux',
                      buildername='android_clang_dbg_recipe',
                      steps=['check licenses (with patch)'],
                      tryserver=True)
   yield step_failure(mastername='tryserver.chromium.linux',
                      buildername='android_clang_dbg_recipe',
                      steps=['check licenses (with patch)',
                             'check licenses (without patch)'],
                      tryserver=True)