Changing multiprocess test SpawnChild to return a struct.

sandbox/mac/sandbox_mac_compiler_unittest.mm

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <fcntl.h>
 #include <stdint.h>
 #include <sys/stat.h>
 #include <unistd.h>
 
 #include "base/process/kill.h"
@@ skipping 14 matching lines @@
 
   SandboxCompiler compiler(profile);
 
   std::string error;
   CHECK(compiler.CompileAndApplyProfile(&error));
 
   return 0;
 }
 
 TEST_F(SandboxMacCompilerTest, BasicProfileTest) {
-  base::Process process = SpawnChild("BasicProfileProcess");
+  base::SpawnChildResult spawn_result = SpawnChild("BasicProfileProcess");
+  base::Process process = std::move(spawn_result.process);
   ASSERT_TRUE(process.IsValid());
   int exit_code = 42;
   EXPECT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_max_timeout(),
                                              &exit_code));
   EXPECT_EQ(exit_code, 0);
 }
 
 MULTIPROCESS_TEST_MAIN(BasicProfileWithParamProcess) {
   std::string profile =
       "(version 1)"
       "(allow file-read* file-write* (literal (param \"DIR\")))";
 
   SandboxCompiler compiler(profile);
   CHECK(compiler.InsertStringParam("DIR", "/"));
 
   std::string error;
   CHECK(compiler.CompileAndApplyProfile(&error));
 
   return 0;
 }
 
 TEST_F(SandboxMacCompilerTest, BasicProfileTestWithParam) {
-  base::Process process = SpawnChild("BasicProfileWithParamProcess");
+  base::SpawnChildResult spawn_result =
+      SpawnChild("BasicProfileWithParamProcess");
+  base::Process process = std::move(spawn_result.process);
   ASSERT_TRUE(process.IsValid());
   int exit_code = 42;
   EXPECT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_max_timeout(),
                                              &exit_code));
   EXPECT_EQ(exit_code, 0);
 }
 
 MULTIPROCESS_TEST_MAIN(ProfileFunctionalProcess) {
   std::string profile =
       "(version 1)"
@@ skipping 10 matching lines @@
   uint8_t byte;
   int fd = open("/dev/urandom", O_RDONLY);
   CHECK_NE(fd, -1);
 
   EXPECT_TRUE(read(fd, &byte, sizeof(byte)) == sizeof(byte));
 
   return 0;
 }
 
 TEST_F(SandboxMacCompilerTest, ProfileFunctionalityTest) {
-  base::Process process = SpawnChild("ProfileFunctionalProcess");
+  base::SpawnChildResult spawn_result = SpawnChild("ProfileFunctionalProcess");
+  base::Process process = std::move(spawn_result.process);
   ASSERT_TRUE(process.IsValid());
   int exit_code = 42;
   EXPECT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_max_timeout(),
                                              &exit_code));
   EXPECT_EQ(exit_code, 0);
 }
 
 MULTIPROCESS_TEST_MAIN(ProfileFunctionalTestWithParamsProcess) {
   std::string profile =
       "(version 1)"
@@ skipping 19 matching lines @@
   EXPECT_TRUE(read(fd, &byte, sizeof(byte)) == sizeof(byte));
 
   // Make sure the sandbox isn't overly permissive.
   struct stat st;
   EXPECT_EQ(stat("/", &st), -1);
 
   return 0;
 }
 
 TEST_F(SandboxMacCompilerTest, ProfileFunctionalityTestWithParams) {
-  base::Process process = SpawnChild("ProfileFunctionalTestWithParamsProcess");
+  base::SpawnChildResult spawn_result =
+      SpawnChild("ProfileFunctionalTestWithParamsProcess");
+  base::Process process = std::move(spawn_result.process);
   ASSERT_TRUE(process.IsValid());
   int exit_code = 42;
   EXPECT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_max_timeout(),
                                              &exit_code));
   EXPECT_EQ(exit_code, 0);
 }
 
 MULTIPROCESS_TEST_MAIN(ProfileFunctionalityTestErrorProcess) {
   std::string profile = "(+ 5 a)";
 
   SandboxCompiler compiler(profile);
 
   // Make sure that this invalid profile results in an error returned.
   std::string error;
   CHECK_EQ(error, "");
   CHECK(!compiler.CompileAndApplyProfile(&error));
   CHECK_NE(error, "");
 
   return 0;
 }
 
 TEST_F(SandboxMacCompilerTest, ProfileFunctionalityTestError) {
-  base::Process process = SpawnChild("ProfileFunctionalityTestErrorProcess");
+  base::SpawnChildResult spawn_result =
+      SpawnChild("ProfileFunctionalityTestErrorProcess");
+  base::Process process = std::move(spawn_result.process);
   ASSERT_TRUE(process.IsValid());
   int exit_code = 42;
   EXPECT_TRUE(process.WaitForExitWithTimeout(TestTimeouts::action_max_timeout(),
                                              &exit_code));
   EXPECT_EQ(exit_code, 0);
 }
 
 }  // namespace sandbox