Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(129)

Side by Side Diff: scripts/slave/recipe_modules/gpu/api.py

Issue 620703002: Add isolate.clean_isolated_files and use in all related recipes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from slave import recipe_api 5 from slave import recipe_api
6 6
7 import common 7 import common
8 8
9 SIMPLE_TESTS_TO_RUN = [ 9 SIMPLE_TESTS_TO_RUN = [
10 'content_gl_tests', 10 'content_gl_tests',
(...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after
156 self.m.chromium.runhooks() 156 self.m.chromium.runhooks()
157 # Since performance tests aren't run on the debug builders, it isn't 157 # Since performance tests aren't run on the debug builders, it isn't
158 # necessary to build all of the targets there. 158 # necessary to build all of the targets there.
159 build_tag = '' if self.m.chromium.is_release_build else 'debug_' 159 build_tag = '' if self.m.chromium.is_release_build else 'debug_'
160 # It's harmless to process the isolate-related targets even if they 160 # It's harmless to process the isolate-related targets even if they
161 # aren't supported on the current configuration (because the component 161 # aren't supported on the current configuration (because the component
162 # build is used). 162 # build is used).
163 is_tryserver = self.m.tryserver.is_tryserver 163 is_tryserver = self.m.tryserver.is_tryserver
164 targets = ['chromium_gpu_%sbuilder' % build_tag] + [ 164 targets = ['chromium_gpu_%sbuilder' % build_tag] + [
165 '%s_run' % test for test in common.GPU_ISOLATES] 165 '%s_run' % test for test in common.GPU_ISOLATES]
166 self.m.isolate.clean_isolated_files(
167 self.m.chromium.c.build_dir.join(self.m.chromium.c.build_config_fs))
166 if is_tryserver: 168 if is_tryserver:
167 try: 169 try:
168 self.m.chromium.compile(targets, name='compile (with patch)') 170 self.m.chromium.compile(targets, name='compile (with patch)')
169 except self.m.step.StepFailure: 171 except self.m.step.StepFailure:
170 if self.m.platform.is_win: 172 if self.m.platform.is_win:
171 self.m.chromium.taskkill() 173 self.m.chromium.taskkill()
172 bot_update_json = self._bot_update.json.output 174 bot_update_json = self._bot_update.json.output
173 self.m.gclient.c.revisions['src'] = str( 175 self.m.gclient.c.revisions['src'] = str(
174 bot_update_json['properties']['got_revision']) 176 bot_update_json['properties']['got_revision'])
175 self.m.bot_update.ensure_checkout(force=True, 177 self.m.bot_update.ensure_checkout(force=True,
(...skipping 212 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 self.get_build_revision(), 390 self.get_build_revision(),
389 self.get_webkit_revision(), 391 self.get_webkit_revision(),
390 args=test_args, 392 args=test_args,
391 name=name, 393 name=name,
392 master_class_name=self._master_class_name_for_testing, 394 master_class_name=self._master_class_name_for_testing,
393 spawn_dbus=True, 395 spawn_dbus=True,
394 **kwargs) 396 **kwargs)
395 except self.m.step.StepFailure: 397 except self.m.step.StepFailure:
396 # Return test name in the event of failure 398 # Return test name in the event of failure
397 return test 399 return test
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/chromium_tests/api.py ('k') | scripts/slave/recipe_modules/isolate/api.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698