Win should call ::TerminateProcess to exit a renderer process

base/process/process_unittest.cc

 // Copyright 2014 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 "base/process/process.h"
 
 #include <utility>
 
 #include "base/process/kill.h"
 #include "base/test/multiprocess_test.h"
@@ skipping 130 matching lines @@
                                  &exit_code);
 
   EXPECT_NE(TERMINATION_STATUS_STILL_RUNNING,
             GetTerminationStatus(process.Handle(), &exit_code));
 #if !defined(OS_POSIX)
   // The POSIX implementation actually ignores the exit_code.
   EXPECT_EQ(kExpectedExitCode, exit_code);
 #endif
 }
 
+TEST_F(ProcessTest, TerminateCurrentProcessImmediately) {
+  Process process(SpawnChild("SleepyChildProcess"));
+  ASSERT_TRUE(process.IsValid());
+
+  const int kDummyExitCode = 42;
+  int exit_code = kDummyExitCode;
+  EXPECT_EQ(TERMINATION_STATUS_STILL_RUNNING,
+            GetTerminationStatus(process.Handle(), &exit_code));
+  EXPECT_EQ(kExpectedStillRunningExitCode, exit_code);
+
+  exit_code = kDummyExitCode;
+  int kExpectedExitCode = 250;
+  process.TerminateCurrentProcessImmediately(kExpectedExitCode);

[Primiano Tucci (use gerrit), 2017/01/16 14:37:34]

well this is the problem. You don't want to terminate the test_f process, but
the child one.
concretely I think you need to add a :
MULTIPROCESS_TEST_MAIN(TerminateImmediatelyChildProcess) {
  Process::TerminateCurrentProcessImmediately(kExpectedExitCode);
  return 0;
}

and here in this funciton you SpawnChild("TerminateImmediatelyChildProcess") and
check that it did exit with kExpectedExitCode

+  process.WaitForExitWithTimeout(TestTimeouts::action_max_timeout(),
+                                 &exit_code);
+
+  EXPECT_NE(TERMINATION_STATUS_STILL_RUNNING,
+            GetTerminationStatus(process.Handle(), &exit_code));
+#if !defined(OS_POSIX)
+  // The POSIX implementation actually ignores the exit_code.

[Primiano Tucci (use gerrit), 2017/01/16 14:37:34]

Where did this happen? I tried both on OSX and Linux and it seems to be
respected. 

+  EXPECT_EQ(kExpectedExitCode, exit_code);
+#endif
+}
+
 MULTIPROCESS_TEST_MAIN(FastSleepyChildProcess) {
   PlatformThread::Sleep(TestTimeouts::tiny_timeout() * 10);
   return 0;
 }
 
 TEST_F(ProcessTest, WaitForExit) {
   Process process(SpawnChild("FastSleepyChildProcess"));
   ASSERT_TRUE(process.IsValid());
 
   const int kDummyExitCode = 42;
@@ skipping 86 matching lines @@
       "2:freezer:/chrome_renderers/to_be_frozen\n"
       "1:cpu:/chrome_renderers/background\n";
 
   EXPECT_FALSE(IsProcessBackgroundedCGroup(kNotBackgrounded));
   EXPECT_TRUE(IsProcessBackgroundedCGroup(kBackgrounded));
 }
 
 #endif  // defined(OS_CHROMEOS)
 
 }  // namespace base