Added code so renderer would cleanly exit.

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 9acb852dff0cea06f7e4d1860194f8ab719217bf..65ec07603e3ca00915006719136a1eab7ba1d1e5 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -196,8 +196,15 @@ class SuicideOnChannelErrorFilter : public IPC::ChannelProxy::MessageFilter {
     //
     // So, we install a filter on the channel so that we can process this event
     // here and kill the process.
-
-    _exit(0);
+    //
+    // 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 (CommandLine::ForCurrentProcess()->
+        HasSwitch(switches::kRendererCleanExit))

[Markus (顧孟勤), 2012/04/09 23:45:46]

I am sure, I am just confused here. Shouldn't we be looking at the
terminate_child_on_shutdown variable that you set in the other object? Is this
variable ever used anywhere?

[asharif1, 2012/04/10 03:08:00]

As far as I understand, that variable is set in the parent browser process and
is unset in the case the command line flag is given. That causes the parent
process not to send a SIGTERM signal to the renderer.

In addition I need this here so _exit() is not called and the renderer has time
to actually shut down properly and call exit() along with the handlers.

I am not sure whether the renderer process uses the same ChildProcessLauncher
class. If so, I may be able to use it here...

+      alarm(30);
+    else
+      _exit(0);
   }
 };
 #endif  // OS_POSIX