Enable isolate on Linux release builder.

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 copy
 
 from slave import recipe_api
 
 
 # Different types of builds this recipe module can do.
@@ skipping 203 matching lines @@
     tests = bot_config.get('tests', [])
     if override_tests is not None:
       tests = override_tests
 
     self.m.chromium.cleanup_temp()
     if self.m.chromium.c.TARGET_PLATFORM == 'android':
       self.m.chromium_android.clean_local_files()
 
     if bot_type in ['builder', 'builder_tester']:
       compile_targets = set(bot_config.get('compile_targets', []))
-      for test in tests:
-        compile_targets.update(test.compile_targets(self.m))
+      tests_including_triggered = copy.copy(tests)

[M-A Ruel, 2014/09/24 19:37:00]

tests_including_triggered = tests[:]

[jam, 2014/09/24 19:47:37]

Done.

       for loop_buildername, builder_dict in master_dict.get(
           'builders', {}).iteritems():
         if builder_dict.get('parent_buildername') == buildername:
           for test in builder_dict.get('tests', []):
-            compile_targets.update(test.compile_targets(self.m))
+            tests_including_triggered.append(test)
+
+      for t in tests_including_triggered:
+        compile_targets.update(t.compile_targets(self.m))
 
       self.m.chromium.compile(targets=sorted(compile_targets))
       self.m.chromium.checkdeps()
 
       if self.m.chromium.c.TARGET_PLATFORM == 'android':
         self.m.chromium_android.check_webview_licenses()
         self.m.chromium_android.findbugs()
 
-      has_swarming_tests = any(t.uses_swarming for t in tests)
-      if bot_config.get('use_isolate'):
-        self.m.isolate.find_isolated_tests(self.m.chromium.output_dir)
-      # TODO(phajdan.jr): Always use the below codepath once fully tested.
-      elif has_swarming_tests:
-        isolated_targets = [t.name for t in tests if t.uses_swarming]
+      isolated_targets = [t.name for t in tests_including_triggered \

[M-A Ruel, 2014/09/24 19:37:00]

isolated_targets = [
  t.name for t in tests_including_triggered if t.uses_swarming
]

[jam, 2014/09/24 19:47:37]

Done.

+          if t.uses_swarming]
+      if isolated_targets:
         self.m.isolate.find_isolated_tests(
             self.m.chromium.output_dir, targets=list(set(isolated_targets)))
 
     got_revision = update_step.presentation.properties['got_revision']
 
     if bot_type == 'builder':
       if mastername == 'chromium.linux':
         # TODO(samuong): This is restricted to Linux for now until I have more
         # confidence that it is not totally broken.
         self.m.archive.archive_dependencies(
@@ skipping 62 matching lines @@
       self.m.chromium.crash_handler()
 
     try:
       return test_runner()
     finally:
       if self.m.platform.is_win:
         self.m.chromium.process_dumps()
 
       if self.m.chromium.c.TARGET_PLATFORM == 'android':
         self.m.chromium_android.common_tests_final_steps()