From acdd8de5e3b0b72e75a7286c6f5dbd0fe425ec85 Mon Sep 17 00:00:00 2001
From: "troy%netscape.com" <troy%netscape.com>
Date: Mon, 10 Aug 1998 18:20:20 +0000
Subject: [PATCH] Added code to delete nsBrowserWindow object

---
 webshell/tests/viewer/nsBrowserWindow.cpp | 2 +-
 webshell/tests/viewer/nsViewerApp.cpp     | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/webshell/tests/viewer/nsBrowserWindow.cpp b/webshell/tests/viewer/nsBrowserWindow.cpp
index f9bdfa93e647..a6c89e27e95d 100644
--- a/webshell/tests/viewer/nsBrowserWindow.cpp
+++ b/webshell/tests/viewer/nsBrowserWindow.cpp
@@ -203,6 +203,7 @@ HandleBrowserEvent(nsGUIEvent *aEvent)
 
   case NS_DESTROY:
     bw->Destroy();
+    NS_RELEASE(bw);
     return nsEventStatus_eConsumeDoDefault;
 
   case NS_MENU_SELECTED:
@@ -471,7 +472,6 @@ nsBrowserWindow::nsBrowserWindow()
 
 nsBrowserWindow::~nsBrowserWindow()
 {
-  Destroy();
 }
 
 NS_IMPL_ADDREF(nsBrowserWindow)
diff --git a/webshell/tests/viewer/nsViewerApp.cpp b/webshell/tests/viewer/nsViewerApp.cpp
index 1f8d4a2ce760..bc414e759c13 100644
--- a/webshell/tests/viewer/nsViewerApp.cpp
+++ b/webshell/tests/viewer/nsViewerApp.cpp
@@ -350,6 +350,9 @@ NS_IMETHODIMP
 nsViewerApp::OpenWindow()
 {
   // Create browser window
+  // XXX Some piece of code needs to properly hold the reference to this
+  // browser window. For the time being the reference is released by the
+  // browser event handling code during processing of the NS_DESTROY event...
   nsBrowserWindow* bw = nsnull;
   nsresult rv = NSRepository::CreateInstance(kBrowserWindowCID, nsnull,
                                              kIBrowserWindowIID,