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

Unified Diff: scripts/slave/recipe_modules/chromium_tests/chromium_webkit.py

Issue 1862773003: Remove references to crash_service, which is no longer needed. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: update expectations Created 4 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium_tests/chromium_webkit.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/chromium_webkit.py b/scripts/slave/recipe_modules/chromium_tests/chromium_webkit.py
index e7724eabef9b3c2ca16c070ae8aa9d2b6186a5a1..5f3c55be7bcd59602d98cdfee75d3bb12023a510 100644
--- a/scripts/slave/recipe_modules/chromium_tests/chromium_webkit.py
+++ b/scripts/slave/recipe_modules/chromium_tests/chromium_webkit.py
@@ -24,11 +24,6 @@ SPEC['builders'].update({
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 32,
},
- 'compile_targets': [
- # TODO(phajdan.jr): Find a way to automatically add crash_service
- # to Windows builds (so that start_crash_service step works).
- 'crash_service',
- ],
'bot_type': 'builder',
'testing': {
'platform': 'win',
@@ -96,10 +91,6 @@ SPEC['builders'].update({
'compile_targets': [
# TODO(phajdan.jr): Shouldn't be needed once we have 64-bit testers.
'blink_tests',
-
- # TODO(phajdan.jr): Find a way to automatically add crash_service
- # to Windows builds (so that start_crash_service step works).
- 'crash_service',
],
'bot_type': 'builder_tester',
'test_generators': [
@@ -121,11 +112,6 @@ SPEC['builders'].update({
'BUILD_CONFIG': 'Debug',
'TARGET_BITS': 32,
},
- 'compile_targets': [
- # TODO(phajdan.jr): Find a way to automatically add crash_service
- # to Windows builds (so that start_crash_service step works).
- 'crash_service',
- ],
'bot_type': 'builder',
'testing': {
'platform': 'win',
@@ -169,10 +155,6 @@ SPEC['builders'].update({
'compile_targets': [
# TODO(phajdan.jr): Shouldn't be needed once we have 64-bit testers.
'blink_tests',
-
- # TODO(phajdan.jr): Find a way to automatically add crash_service
- # to Windows builds (so that start_crash_service step works).
- 'crash_service',
],
'bot_type': 'builder_tester',
'test_generators': [
@@ -547,11 +529,6 @@ SPEC['builders'].update({
'BUILD_CONFIG': 'Release',
'TARGET_BITS': 32,
},
- 'compile_targets': [
- # TODO(phajdan.jr): Find a way to automatically add crash_service
- # to Windows builds (so that start_crash_service step works).
- 'crash_service',
- ],
'test_generators': [
steps.generate_gtest,
steps.generate_script,
@@ -577,11 +554,6 @@ SPEC['builders'].update({
'BUILD_CONFIG': 'Debug',
'TARGET_BITS': 32,
},
- 'compile_targets': [
- # TODO(phajdan.jr): Find a way to automatically add crash_service
- # to Windows builds (so that start_crash_service step works).
- 'crash_service',
- ],
'test_generators': [
steps.generate_gtest,
steps.generate_script,
« no previous file with comments | « scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py ('k') | scripts/slave/recipe_modules/chromium_tests/steps.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698