Adds some debugging code in hopes of isolating a crash.

chrome/browser/ui/views/hung_renderer_view.cc

 // Copyright (c) 2011 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 "chrome/browser/hung_renderer_dialog.h"
 
 #include "base/i18n/rtl.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/renderer_host/render_process_host.h"
@@ skipping 54 matching lines @@
 ///////////////////////////////////////////////////////////////////////////////
 // HungPagesTableModel, public:
 
 HungPagesTableModel::HungPagesTableModel() : observer_(NULL) {
 }
 
 HungPagesTableModel::~HungPagesTableModel() {
 }
 
 void HungPagesTableModel::InitForTabContents(TabContents* hung_contents) {
+  // TODO(sky): remove when figure out cause of 58853.
+  CHECK(hung_contents->render_view_host());
   tab_contentses_.clear();
   for (TabContentsIterator it; !it.done(); ++it) {
+    // TODO(sky): remove when figure out cause of 58853.
+    CHECK(it->render_view_host());
+    CHECK(hung_contents->render_view_host());
     if (it->GetRenderProcessHost() == hung_contents->GetRenderProcessHost())
       tab_contentses_.push_back(*it);
   }
   // The world is different.
   if (observer_)
     observer_->OnModelChanged();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // HungPagesTableModel, views::GroupTableModel implementation:
@@ skipping 132 matching lines @@
       initialized_(false) {
   InitClass();
 }
 
 HungRendererDialogView::~HungRendererDialogView() {
   hung_pages_table_->SetModel(NULL);
 }
 
 void HungRendererDialogView::ShowForTabContents(TabContents* contents) {
   DCHECK(contents && window());
+  // TODO(sky): remove when figure out cause of 58853.
+  CHECK(contents->render_view_host());
   contents_ = contents;
 
   // Don't show the warning unless the foreground window is the frame, or this
   // window (but still invisible). If the user has another window or
   // application selected, activating ourselves is rude.
   HWND frame_hwnd = GetAncestor(contents->GetNativeView(), GA_ROOT);
   HWND foreground_window = GetForegroundWindow();
   if (foreground_window != frame_hwnd &&
       foreground_window != window()->GetNativeWindow()) {
     return;
   }
 
   if (!window()->IsActive()) {
+    // TODO(sky): remove when figure out cause of 58853.
+    CHECK(contents->render_view_host());
+
     gfx::Rect bounds = GetDisplayBounds(contents);
     window()->SetBounds(bounds, frame_hwnd);
 
     // We only do this if the window isn't active (i.e. hasn't been shown yet,
     // or is currently shown but deactivated for another TabContents). This is
     // because this window is a singleton, and it's possible another active
     // renderer may hang while this one is showing, and we don't want to reset
     // the list of hung pages for a potentially unrelated renderer while this
     // one is showing.
     hung_pages_table_model_->InitForTabContents(contents);
@@ skipping 190 matching lines @@
 
 static HungRendererDialogView* CreateHungRendererDialogView() {
   HungRendererDialogView* cv = new HungRendererDialogView;
   views::Window::CreateChromeWindow(NULL, gfx::Rect(), cv);
   return cv;
 }
 
 namespace hung_renderer_dialog {
 
 void ShowForTabContents(TabContents* contents) {
+  // TODO(sky): remove when figure out cause of 58853.
+  CHECK(contents->render_view_host());
   if (!logging::DialogsAreSuppressed()) {
     if (!g_instance)
       g_instance = CreateHungRendererDialogView();
     g_instance->ShowForTabContents(contents);
   }
 }
 
 // static
 void HideForTabContents(TabContents* contents) {
   if (!logging::DialogsAreSuppressed() && g_instance)
     g_instance->EndForTabContents(contents);
 }
 
 }  // namespace hung_renderer_dialog