Fix MojoApplicationHost assertion when navigating from RenderProcessGone.

content/browser/renderer_host/render_process_host_impl.cc

Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index 37e8b336ffba2f0d5bf32283e09a099e1cea4372..9cfc09edd5559d8c8a63a0c15d0464902d3e9f62 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -2049,6 +2049,8 @@ void RenderProcessHostImpl::ProcessDied(bool already_dead) {
 #endif
   RemoveUserData(kSessionStorageHolderKey);
 
+  mojo_application_host_.reset(new MojoApplicationHost);

[Avi (use Gerrit), 2015/01/13 19:20:33]

As a non-mojo type, a comment would be good here. Maybe something like:

// RenderProcessGone handlers might navigate or perform other actions that
require a connection;
// ensure that there is one before calling them.

or the like.

[Marshall, 2015/01/13 19:36:42]

Done.

+
   IDMap<IPC::Listener>::iterator iter(&listeners_);
   while (!iter.IsAtEnd()) {
     iter.GetCurrentValue()->OnMessageReceived(
@@ -2058,8 +2060,6 @@ void RenderProcessHostImpl::ProcessDied(bool already_dead) {
     iter.Advance();
   }
 
-  mojo_application_host_.reset(new MojoApplicationHost);
-
   // It's possible that one of the calls out to the observers might have caused
   // this object to be no longer needed.
   if (delayed_cleanup_needed_)