Make the RenderViewHostImpl update its visibility after a swap.

content/browser/web_contents/render_view_host_manager.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 "content/browser/web_contents/render_view_host_manager.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
@@ skipping 730 matching lines @@
   RenderViewHostImpl* old_render_view_host = render_view_host_;
   render_view_host_ = pending_render_view_host_;
   pending_render_view_host_ = NULL;
 
   // The process will no longer try to exit, so we can decrement the count.
   render_view_host_->GetProcess()->RemovePendingView();
 
   // If the view is gone, then this RenderViewHost died while it was hidden.
   // We ignored the RenderProcessGone call at the time, so we should send it now
   // to make sure the sad tab shows up, etc.
-  if (render_view_host_->GetView())
+  if (render_view_host_->GetView()) {
+    render_view_host_->WasShown();
     render_view_host_->GetView()->Show();

[Fady Samuel, 2013/09/17 20:14:43]

Drive-by thought: It seems that on most platforms (other than Aura), calling
Show calls WasShown(). Maybe the fix here is to have
RenderWidgetHostViewAura::Show call RenderWidgetHostViewAura::WasShown as is the
pattern on other platforms?

+  }
   else
     delegate_->RenderProcessGoneFromRenderManager(render_view_host_);
 
   // Hide the old view now that the new one is visible.
   if (old_render_view_host->GetView()) {
     old_render_view_host->GetView()->Hide();
     old_render_view_host->WasSwappedOut();
   }
 
   // Make sure the size is up to date.  (Fix for bug 1079768.)
@@ skipping 282 matching lines @@
 RenderViewHostImpl* RenderViewHostManager::GetSwappedOutRenderViewHost(
     SiteInstance* instance) {
   RenderViewHostMap::iterator iter = swapped_out_hosts_.find(instance->GetId());
   if (iter != swapped_out_hosts_.end())
     return iter->second;
 
   return NULL;
 }
 
 }  // namespace content