Don't make ServiceChildProcessHost derive from ChildProcessInfo. It didn't really need to. This i...

chrome/service/service_utility_process_host.cc

Index: chrome/service/service_utility_process_host.cc
===================================================================
--- chrome/service/service_utility_process_host.cc	(revision 112096)
+++ chrome/service/service_utility_process_host.cc	(working copy)
@@ -21,14 +21,14 @@
 #if defined(OS_WIN)
 #include "base/memory/scoped_ptr.h"
 #include "base/win/scoped_handle.h"
+#include "content/common/child_process_info.h"

[Scott Byer, 2011/11/30 20:48:33]

nit: include order

[jam, 2011/11/30 20:54:20]

?

 #include "content/common/child_process_messages.h"
 #include "printing/emf_win.h"
 #endif
 
 ServiceUtilityProcessHost::ServiceUtilityProcessHost(
     Client* client, base::MessageLoopProxy* client_message_loop_proxy)
-        : ServiceChildProcessHost(ChildProcessInfo::UTILITY_PROCESS),
-          client_(client),
+        : client_(client),
           client_message_loop_proxy_(client_message_loop_proxy),
           waiting_for_reply_(false) {
   process_id_ = ChildProcessInfo::GenerateChildProcessUniqueId();
@@ -95,10 +95,6 @@
 
 bool ServiceUtilityProcessHost::StartProcess(bool no_sandbox,
                                              const FilePath& exposed_dir) {
-  // Name must be set or metrics_service will crash in any test which
-  // launches a UtilityProcessHost.
-  set_name(ASCIIToUTF16("utility process"));
-
   if (!CreateChannel())
     return false;