Guard against a crash that can impact Dialog boxes ... this one seems to have...

chrome/views/dialog_client_view.cc

 // Copyright (c) 2006-2008 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 <windows.h>
 #include <uxtheme.h>
 #include <vsstyle.h>
 
 #include "chrome/views/dialog_client_view.h"
 
@@ skipping 291 matching lines @@
 bool DialogClientView::AcceleratorPressed(const Accelerator& accelerator) {
   DCHECK(accelerator.GetKeyCode() == VK_ESCAPE);  // We only expect Escape key.
   window()->Close();
   return true;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // DialogClientView, NativeButton::Listener implementation:
 
 void DialogClientView::ButtonPressed(NativeButton* sender) {
+  // We NULL check the delegate here since the buttons can receive WM_COMMAND
+  // messages even after they (and the window containing us) are destroyed.
+  if (GetDialogDelegate())

[Mark Larson, 2009/03/06 00:34:40]

Should this be !GetDialogDelegate() ?

+    return;
+
   if (sender == ok_button_) {
     AcceptWindow();
   } else if (sender == cancel_button_) {
     CancelWindow();
   } else {
     NOTREACHED();
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ skipping 104 matching lines @@
   static bool initialized = false;
   if (!initialized) {
     ResourceBundle& rb = ResourceBundle::GetSharedInstance();
     dialog_button_font_ = rb.GetFont(ResourceBundle::BaseFont);
     initialized = true;
   }
 }
 
 }  // namespace views