Added code to delete nsBrowserWindow object

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)

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,