Fixing bug 199489. Crash when changing iframe visibility from within the iframe. r=adamlock@netscape.com, sr=heikki@netscape.com.

git-svn-id: svn://10.0.0.236/trunk@141401 18797224-902f-48f8-a5cc-f745e15eee43
2003-04-18 18:55:10 +00:00
parent de43e8ea89
commit b8eb00005f
3 changed files with 64 additions and 60 deletions
--- a/mozilla/docshell/base/nsWebShell.cpp
+++ b/mozilla/docshell/base/nsWebShell.cpp
@@ -252,7 +252,10 @@ nsWebShell::EnsureCommandHandler()
    if (!commandUpdater) return NS_ERROR_FAILURE;
    
    nsCOMPtr<nsIDOMWindow> domWindow = do_GetInterface(NS_STATIC_CAST(nsIInterfaceRequestor *, this));
-    nsresult rv = commandUpdater->Init(domWindow);
+#ifdef DEBUG
+    nsresult rv =
+#endif
+    commandUpdater->Init(domWindow);
    NS_ASSERTION(NS_SUCCEEDED(rv), "Initting command manager failed");
  }
  
@@ -569,7 +572,6 @@ nsWebShell::OnLinkClickSync(nsIContent *aContent,
                            nsIDocShell** aDocShell,
                            nsIRequest** aRequest)
 {
-  nsresult rv;
  nsAutoString target(aTargetSpec);

  // Initialize the DocShell / Request