Assume crashed RenderWidgetHost to be invisible

content/browser/frame_host/render_frame_host_manager.cc

Index: content/browser/frame_host/render_frame_host_manager.cc
diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
index c2ef320311620f6587e9483f41ed7abcb146846c..5175678c81c32c3b92701d5233daa4e42c08fcda 100644
--- a/content/browser/frame_host/render_frame_host_manager.cc
+++ b/content/browser/frame_host/render_frame_host_manager.cc
@@ -427,6 +427,17 @@ RenderFrameHostImpl* RenderFrameHostManager::Navigate(
       if (dest_render_frame_host->GetView())
         dest_render_frame_host->GetView()->Hide();
     } else {
+#if defined(OS_ANDROID)

[no sievers, 2015/09/01 18:30:54]

Can we remove the ifdef Android?
There shouldn't be a difference in behavior. And 'defined(AURA) &&
defined(OS_ANDROID)' needs to be working also.

You can try on AURA Linux and see if that works for crashed tabs.

In other words I was wondering if the fix could just be to 'remove the
visibility workaround for when the renderer crashes, and instead set the
visibility correctly, according to what the app thinks the visibility should be,
when we reload the crashed tab'.

[Jaekyun Seok (inactive), 2015/09/02 01:25:49]

Removed.

I tested on AURA Linux manually, and confirmed that it worked.

+      if (dest_render_frame_host->GetView()) {
+        if (dest_render_frame_host->GetView()->IsShowing() ==
+            delegate_->IsHidden()) {
+          if (delegate_->IsHidden())
+            dest_render_frame_host->GetView()->Hide();
+          else
+            dest_render_frame_host->GetView()->Show();
+        }
+      }
+#endif
       // TODO(nasko): This is a very ugly hack. The Chrome extensions process
       // manager still uses NotificationService and expects to see a
       // RenderViewHost changed notification after WebContents and