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

content/public/browser/render_process_host.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_
 
 #include "base/basictypes.h"
 #include "base/id_map.h"
 #include "base/process/kill.h"
@@ skipping 84 matching lines @@
   // other non-guest renderers in the same process if IsIsolatedGuest is false.
   virtual bool IsIsolatedGuest() const = 0;
 
   // Returns the storage partition associated with this process.
   //
   // TODO(nasko): Remove this function from the public API once
   // URLRequestContextGetter's creation is moved into StoragePartition.
   // http://crbug.com/158595
   virtual StoragePartition* GetStoragePartition() const = 0;
 
+  // Try to shutdown the associated renderer process forcedly, giving it the

[Charlie Reis, 2014/12/02 17:13:37]

nit: shut down
nit: drop "forcedly," instead say "without running unload handlers, etc"

[Jaekyun Seok (inactive), 2014/12/03 02:51:35]

Done.

+  // 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 shutdown 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.

[Charlie Reis, 2014/12/02 17:13:37]

Might as well drop the "current implementation" sentence, since it's stale.

[Jaekyun Seok (inactive), 2014/12/03 02:51:35]

Done.

-  // Returns True if it was able to do fast shutdown.
+  // Returns true if it was able to do fast shutdown.
   virtual bool FastShutdownIfPossible() = 0;
 
   // Returns true if fast shutdown was started for the renderer.
   virtual bool FastShutdownStarted() const = 0;
 
   // Dump the child process' handle table before shutting down.
   virtual void DumpHandles() = 0;
 
   // Returns the process object associated with the child process.  In certain
   // tests or single-process mode, this will actually represent the current
@@ skipping 168 matching lines @@
   static void SetMaxRendererProcessCount(size_t count);
 
   // Returns the current maximum number of renderer process hosts kept by the
   // content module.
   static size_t GetMaxRendererProcessCount();
 };
 
 }  // namespace content.
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_