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

content/browser/renderer_host/mock_render_process_host.cc

 // 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.
 
 #include "content/browser/renderer_host/mock_render_process_host.h"
 
 #include "content/browser/child_process_security_policy.h"
 
 MockRenderProcessHost::MockRenderProcessHost(
     content::BrowserContext* browser_context)
@@ skipping 56 matching lines @@
 }
 
 
 bool MockRenderProcessHost::FastShutdownIfPossible() {
   // We aren't actually going to do anything, but set |fast_shutdown_started_|
   // to true so that tests know we've been called.
   fast_shutdown_started_ = true;
   return true;
 }
 
+void MockRenderProcessHost::DumpHandles() {
+  return;

[jam, 2011/09/19 17:10:48]

nit: not needed

[Cris Neckar, 2011/09/20 01:22:15]

Done.

+}
+
 base::ProcessHandle MockRenderProcessHost::GetHandle() {
   return base::kNullProcessHandle;
 }
 
 bool MockRenderProcessHost::Send(IPC::Message* msg) {
   // Save the message in the sink.
   sink_.OnMessageReceived(*msg);
   delete msg;
   return true;
 }
@@ skipping 52 matching lines @@
 
 void MockRenderProcessHostFactory::Remove(MockRenderProcessHost* host) const {
   for (ScopedVector<MockRenderProcessHost>::iterator it = processes_.begin();
        it != processes_.end(); ++it) {
     if (*it == host) {
       processes_.weak_erase(it);
       break;
     }
   }
 }