Use StopChildProcess instead of base::KillProcess to kill a renderer process

content/public/browser/render_process_host.h

Index: content/public/browser/render_process_host.h
diff --git a/content/public/browser/render_process_host.h b/content/public/browser/render_process_host.h
index dcec7d8cd8495901142e426a95e81087abbbaeaf..61ecbcf4f46e983adb6635a56e6d653005cc3b83 100644
--- a/content/public/browser/render_process_host.h
+++ b/content/public/browser/render_process_host.h
@@ -99,10 +99,16 @@ class CONTENT_EXPORT RenderProcessHost : public IPC::Sender,
   // http://crbug.com/158595
   virtual StoragePartition* GetStoragePartition() const = 0;
 
-  // Try to shutdown the associated renderer process as fast as possible.
+  // Try to shut down the associated renderer process without running unload
+  // handlers, etc, giving it the specified exit code. If |wait| is true, wait
+  // for the process to be actually terminated before returning.
+  // Returns true if it was able to shut down.
+  virtual bool Shutdown(int exit_code, bool wait) = 0;
+
+  // Try to shut down the associated renderer process as fast as possible.
   // If this renderer has any RenderViews with unload handlers, then this
-  // function does nothing.  The current implementation uses TerminateProcess.
-  // Returns True if it was able to do fast shutdown.
+  // function does nothing.  This function uses TerminateProcess.

[Charlie Reis, 2014/12/03 17:41:37]

Ah, I meant drop the whole sentence.  It doesn't use TerminateProcess anymore,
and we don't need to expose the implementation details in the public interface
comment.

[Jaekyun Seok (inactive), 2014/12/04 01:30:04]

Done.

+  // Returns true if it was able to do fast shutdown.
   virtual bool FastShutdownIfPossible() = 0;
 
   // Returns true if fast shutdown was started for the renderer.