From 9bfd1743c26f2264fcf1951c1a46bd04885c55ba Mon Sep 17 00:00:00 2001
From: "kipp%netscape.com" <kipp%netscape.com>
Date: Tue, 15 Sep 1998 00:16:50 +0000
Subject: [PATCH] Use nsBrowserWindow::CloseAllWindows to shutdown cleanly;
 added a missing NS_RELEASE and removed an improper NS_ADDREF

---
 webshell/tests/viewer/nsViewerApp.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/webshell/tests/viewer/nsViewerApp.cpp b/webshell/tests/viewer/nsViewerApp.cpp
index 324d3349e493..d6d443b12fa2 100644
--- a/webshell/tests/viewer/nsViewerApp.cpp
+++ b/webshell/tests/viewer/nsViewerApp.cpp
@@ -113,6 +113,10 @@ nsViewerApp::QueryInterface(REFNSIID aIID, void** aInstancePtrResult)
 void
 nsViewerApp::Destroy()
 {
+  // Close all of our windows
+  nsBrowserWindow::CloseAllWindows();
+
+  // Release the crawler
   NS_IF_RELEASE(mCrawler);
 
   // Only shutdown if Initialize has been called...
@@ -440,6 +444,7 @@ nsViewerApp::OpenWindow()
   else {
     bw->GoTo(mStartURL);
   }
+  NS_RELEASE(bw);
 
   return NS_OK;
 }
@@ -456,7 +461,6 @@ nsViewerApp::OpenWindow(PRUint32 aNewChromeMask, nsIBrowserWindow*& aNewWindow)
   bw->Init(mAppShell, mPrefs, nsRect(0, 0, 620, 400), aNewChromeMask, mAllowPlugins);
 
   aNewWindow = bw;
-  NS_ADDREF(bw);
 
   return NS_OK;
 }