Index: scripts/slave/recipe_modules/chromium/steps.py |
diff --git a/scripts/slave/recipe_modules/chromium/steps.py b/scripts/slave/recipe_modules/chromium/steps.py |
index f7ea3f5cdd74ded59f0652324b9fc43f06058138..5a5fa86c9ef7f439928c9bbf8e8082c541537a09 100644 |
--- a/scripts/slave/recipe_modules/chromium/steps.py |
+++ b/scripts/slave/recipe_modules/chromium/steps.py |
@@ -98,29 +98,19 @@ |
All new tests are strongly encouraged to use this infrastructure. |
""" |
- def __init__(self, name, script, all_compile_targets): |
+ def __init__(self, name, script): |
super(ScriptTest, self).__init__() |
self._name = name |
self._script = script |
- self._all_compile_targets = all_compile_targets |
@property |
def name(self): |
return self._name |
- def compile_targets(self, api): |
- try: |
- return self._all_compile_targets[self._script] |
- except KeyError: |
- # Not all scripts will have test data inside recipes, |
- # so return a default value. |
- # TODO(phajdan.jr): Revisit this when all script tests |
- # lists move src-side. We should be able to provide |
- # test data then. |
- if api.chromium._test_data.enabled: |
- return [] |
- |
- raise |
+ @staticmethod |
+ def compile_targets(_): |
+ # TODO(phajdan.jr): Implement this (http://crbug.com/422235). |
+ return [] |
def run(self, api, suffix): |
name = self.name |
@@ -133,8 +123,7 @@ |
# for consistency. |
api.path['checkout'].join( |
'testing', 'scripts', api.path.basename(self._script)), |
- args=(api.chromium.get_common_args_for_scripts() + |
- ['run', '--output', api.json.output()]), |
+ args=['run', '--output', api.json.output()], |
step_test_data=lambda: api.json.test_api.output( |
{'valid': True, 'failures': []})) |
finally: |