Move handle dumpage to the renderer process (so that it works correctly) and factor out redundant...

content/browser/renderer_host/render_process_host.h

 // Copyright (c) 2011 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_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_H_
 #pragma once
 
 #include <set>
 
@@ skipping 86 matching lines @@
   void mark_is_extension_process() { is_extension_process_ = true; }
 
   // Used for refcounting, each holder of this object must Attach and Release
   // just like it would for a COM object. This object should be allocated on
   // the heap; when no listeners own it any more, it will delete itself.
   void Attach(IPC::Channel::Listener* listener, int routing_id);
 
   // See Attach()
   void Release(int listener_id);
 
+  // Schedules the host for deletion and removes it from the all_hosts list.
+  void Cleanup();
+
   // Listeners should call this when they've sent a "Close" message and
   // they're waiting for a "Close_ACK", so that if the renderer process
   // goes away we'll know that it was intentional rather than a crash.
   void ReportExpectingClose(int32 listener_id);
 
   // Track the count of pending views that are being swapped back in.  Called
   // by listeners to register and unregister pending views to prevent the
   // process from exiting.
   void AddPendingView();
   void RemovePendingView();
@@ skipping 89 matching lines @@
   virtual void WidgetRestored() = 0;
   virtual void WidgetHidden() = 0;
   virtual int VisibleWidgetCount() const = 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.
   // Returns True if it was able to do fast shutdown.
   virtual bool FastShutdownIfPossible() = 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
   // process.
   //
   // NOTE: this is not necessarily valid immediately after calling Init, as
   // Init starts the process asynchronously.  It's guaranteed to be valid after
   // the first IPC arrives.
   virtual base::ProcessHandle GetHandle() = 0;
 
   // Transport DIB functions ---------------------------------------------------
@@ skipping 113 matching lines @@
 // Factory object for RenderProcessHosts. Using this factory allows tests to
 // swap out a different one to use a TestRenderProcessHost.
 class RenderProcessHostFactory {
  public:
   virtual ~RenderProcessHostFactory() {}
   virtual RenderProcessHost* CreateRenderProcessHost(
       content::BrowserContext* browser_context) const = 0;
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_H_