Assume crashed RenderWidgetHost to be invisible

content/browser/renderer_host/render_widget_host_impl.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_impl.h"
 
 #include <math.h>
 #include <set>
 #include <utility>
 
@@ skipping 1224 matching lines @@
   ResetSizeAndRepaintPendingFlags();
   current_size_.SetSize(0, 0);
   // After the renderer crashes, the view is destroyed and so the
   // RenderWidgetHost cannot track its visibility anymore. We assume such
   // RenderWidgetHost to be visible for the sake of internal accounting - be
   // careful about changing this - see http://crbug.com/401859.
   //
   // We need to at least make sure that the RenderProcessHost is notified about
   // the |is_hidden_| change, so that the renderer will have correct visibility
   // set when respawned.
+  //
+  // http://crbug.com/522795 : On Android, we assume dead RenderWidgetHost to be
+  // invisible to correctly keep the priority of a restored renderer.
+#if defined(OS_ANDROID)

[no sievers, 2015/09/04 16:18:03]

We don't need the ifdef, since the your change for restoring the tab works
regardless of the platform, right?

[Jaekyun Seok (inactive), 2015/09/07 07:40:04]

Done.

+  if (!is_hidden_) {
+    process_->WidgetHidden();
+    is_hidden_ = true;
+  }
+#else
   if (is_hidden_) {
     process_->WidgetRestored();
     is_hidden_ = false;
   }
+#endif
 
   // Reset this to ensure the hung renderer mechanism is working properly.
   in_flight_event_count_ = 0;
   StopHangMonitorTimeout();
 
   if (view_) {
     GpuSurfaceTracker::Get()->SetSurfaceHandle(surface_id_,
                                                gfx::GLSurfaceHandle());
     view_->RenderProcessGone(status, exit_code);
     view_ = NULL; // The View should be deleted by RenderProcessGone.
@@ skipping 869 matching lines @@
 }
 
 #if defined(OS_WIN)
 gfx::NativeViewAccessible
     RenderWidgetHostImpl::GetParentNativeViewAccessible() {
   return delegate_ ? delegate_->GetParentNativeViewAccessible() : NULL;
 }
 #endif
 
 }  // namespace content