From b53d9278820cfd83c25245502e609e1cf4913a93 Mon Sep 17 00:00:00 2001
From: "tbogard%aol.net" <tbogard%aol.net>
Date: Tue, 8 Feb 2000 18:45:55 +0000
Subject: [PATCH] We were checking against TRUE on the return rather than
 NS_SUCCEEDED().  Fix for bug #26974.  Good catch by Putterman.

---
 xpfe/appshell/src/nsXULWindow.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xpfe/appshell/src/nsXULWindow.cpp b/xpfe/appshell/src/nsXULWindow.cpp
index 91f18044918..856dd49ac45 100644
--- a/xpfe/appshell/src/nsXULWindow.cpp
+++ b/xpfe/appshell/src/nsXULWindow.cpp
@@ -776,7 +776,7 @@ NS_IMETHODIMP nsXULWindow::ShowModal()
 
    nsCOMPtr<nsIJSContextStack> stack(do_GetService("nsThreadJSContextStack"));
    nsresult rv = NS_OK;
-   if(stack  && stack->Push(nsnull))
+   if(stack  && NS_SUCCEEDED(stack->Push(nsnull)))
       {
       while(NS_SUCCEEDED(rv) && mContinueModalLoop)
          {