Make the RenderViewHostImpl update its visibility after a swap.

content/browser/renderer_host/render_widget_host_view_android.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/renderer_host/render_widget_host_view_android.h"
 
 #include <android/bitmap.h>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
@@ skipping 337 matching lines @@
 bool RenderWidgetHostViewAndroid::IsSurfaceAvailableForCopy() const {
   return HasValidFrame();
 }
 
 void RenderWidgetHostViewAndroid::Show() {
   if (are_layers_attached_)
     return;
 
   are_layers_attached_ = true;
   AttachLayers();
+
+  WasShown();

[jamesr, 2013/09/18 18:06:53]

render_widget_host_view_win.cc has this comment:

  // |render_widget_host_| may be NULL if the WebContentsImpl is in the process
  // of closing.

and a null-check before it dereferences render_widget_host_. 
RWHVAndroid::WasShown() doesn't do a null check.  one of these is wrong, do you
know which?  RWHVMac doesn't do a null check

 }
 
 void RenderWidgetHostViewAndroid::Hide() {

[jamesr, 2013/09/18 18:06:53]

RWHVWin and RWHVMac calls WasHidden() too, you probably should

   if (!are_layers_attached_)
     return;
 
   are_layers_attached_ = false;
   RemoveLayers();
 }
 
 bool RenderWidgetHostViewAndroid::IsShowing() {
   // ContentViewCoreImpl represents the native side of the Java
   // ContentViewCore.  It being NULL means that it is not attached
@@ skipping 932 matching lines @@
 // RenderWidgetHostView, public:
 
 // static
 RenderWidgetHostView*
 RenderWidgetHostView::CreateViewForWidget(RenderWidgetHost* widget) {
   RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(widget);
   return new RenderWidgetHostViewAndroid(rwhi, NULL);
 }
 
 } // namespace content