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

Unified Diff: scripts/slave/recipes/chromium_trybot.expected/use_v8_patch_on_chromium_trybot.json

Issue 1871583004: Revert "Remove references to crash_service, which is no longer needed." (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: 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/recipes/chromium_trybot.expected/use_v8_patch_on_chromium_trybot.json
diff --git a/scripts/slave/recipes/chromium_trybot.expected/use_v8_patch_on_chromium_trybot.json b/scripts/slave/recipes/chromium_trybot.expected/use_v8_patch_on_chromium_trybot.json
index cbef43094f8153e1abaa25d9842095c2ebdf56c9..7da90fb210831b228672f77a18e909bf03d9a6ec 100644
--- a/scripts/slave/recipes/chromium_trybot.expected/use_v8_patch_on_chromium_trybot.json
+++ b/scripts/slave/recipes/chromium_trybot.expected/use_v8_patch_on_chromium_trybot.json
@@ -259,7 +259,7 @@
"win_chromium_rel_ng",
"-v",
"//out/Release",
- "{\"additional_compile_targets\": [\"chromium_builder_tests\"], \"files\": [\"v8/foo.cc\"], \"test_targets\": [\"blink_heap_unittests\", \"blink_platform_unittests\", \"mini_installer\", \"webkit_unit_tests\", \"wtf_unittests\"]}",
+ "{\"additional_compile_targets\": [\"chromium_builder_tests\", \"crash_service\"], \"files\": [\"v8/foo.cc\"], \"test_targets\": [\"blink_heap_unittests\", \"blink_platform_unittests\", \"mini_installer\", \"webkit_unit_tests\", \"wtf_unittests\"]}",
"/path/to/tmp/json"
],
"cwd": "[SLAVE_BUILD]",
@@ -274,7 +274,8 @@
"@@@STEP_LOG_END@json.output@@@",
"@@@STEP_LOG_LINE@analyze_details@{@@@",
"@@@STEP_LOG_LINE@analyze_details@ \"additional_compile_targets\": [@@@",
- "@@@STEP_LOG_LINE@analyze_details@ \"chromium_builder_tests\"@@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"chromium_builder_tests\", @@@",
+ "@@@STEP_LOG_LINE@analyze_details@ \"crash_service\"@@@",
"@@@STEP_LOG_LINE@analyze_details@ ], @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"compile_targets\": [], @@@",
"@@@STEP_LOG_LINE@analyze_details@ \"self.m.filter.compile_targets\": [], @@@",
@@ -371,6 +372,17 @@
"cmd": [
"python",
"-u",
+ "RECIPE_PACKAGE_REPO[build]\\scripts\\slave\\chromium\\run_crash_handler.py",
+ "--target",
+ "Release"
+ ],
+ "cwd": "[SLAVE_BUILD]",
+ "name": "start_crash_service"
+ },
+ {
+ "cmd": [
+ "python",
+ "-u",
"[SLAVE_BUILD]\\src\\testing\\scripts\\webkit_lint.py",
"--build-config-fs",
"Release",

Powered by Google App Engine
This is Rietveld 408576698