Don't make classes derive from ChildProcessHost, and instead have them use it through composition...

content/browser/worker_host/worker_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/worker_host/worker_process_host.h"
 
 #include <set>
 #include <vector>
 
 #include "base/base_switches.h"
@@ skipping 13 matching lines @@
 #include "content/browser/renderer_host/database_message_filter.h"
 #include "content/browser/renderer_host/file_utilities_message_filter.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/renderer_host/render_view_host_delegate.h"
 #include "content/browser/renderer_host/socket_stream_dispatcher_host.h"
 #include "content/browser/resource_context.h"
 #include "content/browser/user_metrics.h"
 #include "content/browser/worker_host/message_port_service.h"
 #include "content/browser/worker_host/worker_message_filter.h"
 #include "content/browser/worker_host/worker_service.h"
+#include "content/common/child_process_host.h"
 #include "content/common/debug_flags.h"
 #include "content/common/view_messages.h"
 #include "content/common/worker_messages.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/result_codes.h"
 #include "ipc/ipc_switches.h"
 #include "net/base/mime_util.h"
 #include "net/base/registry_controlled_domain.h"
@@ skipping 60 matching lines @@
                      parent_iter->render_view_id()));
     }
     WorkerService::GetInstance()->NotifyWorkerDestroyed(this,
                                                         i->worker_route_id());
   }
 
   ChildProcessSecurityPolicy::GetInstance()->Remove(id());
 }
 
 bool WorkerProcessHost::Init(int render_process_id) {
-  if (!CreateChannel())
+  if (!child_process_host()->CreateChannel())
     return false;
 
 #if defined(OS_LINUX)
-  int flags = CHILD_ALLOW_SELF;
+  int flags = ChildProcessHost::CHILD_ALLOW_SELF;
 #else
-  int flags = CHILD_NORMAL;
+  int flags = ChildProcessHost::CHILD_NORMAL;
 #endif
 
-  FilePath exe_path = GetChildPath(flags);
+  FilePath exe_path = ChildProcessHost::GetChildPath(flags);
   if (exe_path.empty())
     return false;
 
   CommandLine* cmd_line = new CommandLine(exe_path);
   cmd_line->AppendSwitchASCII(switches::kProcessType, switches::kWorkerProcess);
-  cmd_line->AppendSwitchASCII(switches::kProcessChannelID, channel_id());
+  cmd_line->AppendSwitchASCII(switches::kProcessChannelID,
+                              child_process_host()->channel_id());
   std::string locale =
       content::GetContentClient()->browser()->GetApplicationLocale();
   cmd_line->AppendSwitchASCII(switches::kLang, locale);
 
   static const char* const kSwitchNames[] = {
     switches::kWebWorkerShareProcesses,
     switches::kDisableApplicationCache,
     switches::kDisableDatabases,
     switches::kEnableLogging,
     switches::kLoggingLevel,
@@ skipping 90 matching lines @@
 
 void WorkerProcessHost::CreateMessageFilters(int render_process_id) {
   DCHECK(resource_context_);
   net::URLRequestContext* request_context =
       resource_context_->request_context();
 
   ResourceMessageFilter* resource_message_filter = new ResourceMessageFilter(
       id(), content::PROCESS_TYPE_WORKER, resource_context_,
       new URLRequestContextSelector(request_context),
       resource_dispatcher_host_);
-  AddFilter(resource_message_filter);
+  child_process_host()->AddFilter(resource_message_filter);
 
   worker_message_filter_ = new WorkerMessageFilter(
       render_process_id, resource_context_, resource_dispatcher_host_,
       base::Bind(&WorkerService::next_worker_route_id,
                  base::Unretained(WorkerService::GetInstance())));
-  AddFilter(worker_message_filter_);
-  AddFilter(new AppCacheDispatcherHost(
+  child_process_host()->AddFilter(worker_message_filter_);
+  child_process_host()->AddFilter(new AppCacheDispatcherHost(
       resource_context_->appcache_service(), id()));
-  AddFilter(new FileSystemDispatcherHost(
+  child_process_host()->AddFilter(new FileSystemDispatcherHost(
       request_context, resource_context_->file_system_context()));
-  AddFilter(new FileUtilitiesMessageFilter(id()));
-  AddFilter(new BlobMessageFilter(
+  child_process_host()->AddFilter(new FileUtilitiesMessageFilter(id()));
+  child_process_host()->AddFilter(new BlobMessageFilter(
       id(), resource_context_->blob_storage_context()));
-  AddFilter(new MimeRegistryMessageFilter());
-  AddFilter(new DatabaseMessageFilter(
+  child_process_host()->AddFilter(new MimeRegistryMessageFilter());
+  child_process_host()->AddFilter(new DatabaseMessageFilter(
       resource_context_->database_tracker()));
 
   SocketStreamDispatcherHost* socket_stream_dispatcher_host =
       new SocketStreamDispatcherHost(
           new URLRequestContextSelector(request_context), resource_context_);
-  AddFilter(socket_stream_dispatcher_host);
-  AddFilter(new content::WorkerDevToolsMessageFilter(id()));
+  child_process_host()->AddFilter(socket_stream_dispatcher_host);
+  child_process_host()->AddFilter(new content::WorkerDevToolsMessageFilter(id()));
 }
 
 void WorkerProcessHost::CreateWorker(const WorkerInstance& instance) {
   ChildProcessSecurityPolicy::GetInstance()->GrantRequestURL(
       id(), instance.url());
 
   instances_.push_back(instance);
 
   WorkerProcessMsg_CreateWorker_Params params;
   params.url = instance.url();
@@ skipping 350 matching lines @@
     }
   }
   return false;
 }
 
 WorkerProcessHost::WorkerInstance::FilterInfo
 WorkerProcessHost::WorkerInstance::GetFilter() const {
   DCHECK(NumFilters() == 1);
   return *filters_.begin();
 }