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

Unified Diff: components/crash/content/app/fallback_crash_handler_win_unittest.cc

Issue 2733323002: Changing multiprocess test SpawnChild to return a struct. (Closed)
Patch Set: Fixed bots. Created 3 years, 9 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: components/crash/content/app/fallback_crash_handler_win_unittest.cc
diff --git a/components/crash/content/app/fallback_crash_handler_win_unittest.cc b/components/crash/content/app/fallback_crash_handler_win_unittest.cc
index 13ca131f0204c80ae3b82f4d5e95d0649be88b0d..3a088b2584a90a2cedc73e5d8b50b98268fedcd7 100644
--- a/components/crash/content/app/fallback_crash_handler_win_unittest.cc
+++ b/components/crash/content/app/fallback_crash_handler_win_unittest.cc
@@ -169,8 +169,9 @@ TEST_F(FallbackCrashHandlerWinTest, GenerateCrashDump) {
cmd_line.AppendSwitchPath("directory", database_dir_.GetPath());
base::LaunchOptions options;
options.start_hidden = true;
- base::Process test_child = base::SpawnMultiProcessTestChild(
+ base::SpawnChildResult spawn_result = base::SpawnMultiProcessTestChild(
"FallbackCrashHandlerWinMain", cmd_line, options);
+ base::Process test_child = std::move(spawn_result.process);
ASSERT_TRUE(test_child.IsValid());
int exit_code = -1;

Powered by Google App Engine
This is Rietveld 408576698