From deab31ecaa1f90cdf795c728f600e1939d3c01fa Mon Sep 17 00:00:00 2001
From: "vidur%netscape.com" <vidur%netscape.com>
Date: Tue, 15 Jun 1999 03:15:37 +0000
Subject: [PATCH] Fixed memory leak when new windows were opened

---
 webshell/tests/viewer/nsBrowserWindow.cpp | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/webshell/tests/viewer/nsBrowserWindow.cpp b/webshell/tests/viewer/nsBrowserWindow.cpp
index 06ce4bd1bd2..d8919340ccb 100644
--- a/webshell/tests/viewer/nsBrowserWindow.cpp
+++ b/webshell/tests/viewer/nsBrowserWindow.cpp
@@ -1785,21 +1785,22 @@ nsBrowserWindow::FindWebShellWithName(const PRUnichar* aName, nsIWebShell*& aRes
 
   for (i = 0; i < n; i++) {
     nsBrowserWindow* bw = (nsBrowserWindow*) gBrowsers.ElementAt(i);
-    nsIWebShell *ws;
+    nsCOMPtr<nsIWebShell> ws;
     
-    if (NS_OK == bw->GetWebShell(ws)) {
+    if (NS_OK == bw->GetWebShell(*getter_AddRefs(ws))) {
       const PRUnichar *name;
       if (NS_OK == ws->GetName(&name)) {
-	if (aNameStr.Equals(name)) {
-	  aResult = ws;
-	  NS_ADDREF(aResult);
-	  return NS_OK;
-	}
+        if (aNameStr.Equals(name)) {
+          aResult = ws;
+          NS_ADDREF(aResult);
+          return NS_OK;
+        }
       }      
-    }
-    if (NS_OK == ws->FindChildWithName(aName, aResult)) {
-      if (nsnull != aResult) {
-	return NS_OK;
+
+      if (NS_OK == ws->FindChildWithName(aName, aResult)) {
+        if (nsnull != aResult) {
+          return NS_OK;
+        }
       }
     }
   }