On Posix, make all child processes quit when the browser dies, not just the renderers.

content/renderer/renderer_main.cc

 // 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.
 
 #include "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/debug/debugger.h"
 #include "base/debug/trace_event.h"
 #include "base/hi_res_timer_manager.h"
 #include "base/i18n/rtl.h"
@@ skipping 52 matching lines @@
     mach_error_t err = mach_override_ptr(
         (void*)&TISCreateInputSourceList,
         (void*)&ChromeTISCreateInputSourceList,
         NULL);
     CHECK_EQ(err_none, err);
   }
 }
 
 #endif  // OS_MACOSX
 
-#if defined(OS_POSIX)
-
-class SuicideOnChannelErrorFilter : public IPC::ChannelProxy::MessageFilter {
- public:
-  // IPC::ChannelProxy::MessageFilter
-  virtual void OnChannelError() OVERRIDE {
-    // On POSIX, at least, one can install an unload handler which loops
-    // forever and leave behind a renderer process which eats 100% CPU forever.
-    //
-    // This is because the terminate signals (ViewMsg_ShouldClose and the error
-    // from the IPC channel) are routed to the main message loop but never
-    // processed (because that message loop is stuck in V8).
-    //
-    // One could make the browser SIGKILL the renderers, but that leaves open a
-    // large window where a browser failure (or a user, manually terminating
-    // the browser because "it's stuck") will leave behind a process eating all
-    // the CPU.
-    //
-    // So, we install a filter on the channel so that we can process this event
-    // here and kill the process.
-    //
-    // We want to kill this process after giving it 30 seconds to run the exit
-    // handlers. SIGALRM has a default disposition of terminating the
-    // application.
-#if defined(OS_POSIX)
-    if (CommandLine::ForCurrentProcess()->
-        HasSwitch(switches::kRendererCleanExit))
-      alarm(30);
-    else
-#endif
-      _exit(0);
-  }
-
- protected:
-  virtual ~SuicideOnChannelErrorFilter() {}
-};
-
-#endif  // OS(POSIX)
-
 }  // namespace
 
 // This function provides some ways to test crash and assertion handling
 // behavior of the renderer.
 static void HandleRendererErrorTestParameters(const CommandLine& command_line) {
   if (command_line.HasSwitch(switches::kWaitForDebugger))
     base::debug::WaitForDebugger(60, true);
 
   if (command_line.HasSwitch(switches::kRendererStartupDialog))
     ChildProcess::WaitForDebugger("Renderer");
@@ skipping 117 matching lines @@
     if (!no_sandbox) {
       run_loop = platform.EnableSandbox();
     } else {
       LOG(ERROR) << "Running without renderer sandbox";
     }
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
     RenderProcessImpl render_process;
     new RenderThreadImpl();
 #endif
 
-#if defined(OS_POSIX)
-    RenderThreadImpl::current()->AddFilter(new SuicideOnChannelErrorFilter());
-#endif
-
     platform.RunSandboxTests();
 
     startup_timer.Stop();  // End of Startup Time Measurement.
 
     if (run_loop) {
 #if defined(OS_MACOSX)
       if (pool)
         pool->Recycle();
 #endif
       TRACE_EVENT_BEGIN_ETW("RendererMain.START_MSG_LOOP", 0, 0);
       MessageLoop::current()->Run();
       TRACE_EVENT_END_ETW("RendererMain.START_MSG_LOOP", 0, 0);
     }
   }
   platform.PlatformUninitialize();
   TRACE_EVENT_END_ETW("RendererMain", 0, "");
   return 0;
 }