Release mode now compiles and links.

We are getting too many ifdef NS_DEBUG strings in the code and
are ignoring them.  We should stop doing this.

xpfe/xpviewer/src/nsBrowserWindow.cpp

@@ -3171,6 +3171,8 @@ nsBrowserWindow::DoSiteWalker()
   mApp->CreateSiteWalker(this);
 }
 
+#endif NS_DEBUG
+
 //-----------------------------------------------------
 //-- Menu Struct
 //-----------------------------------------------------
@@ -3234,6 +3236,7 @@ nsIMenuItem * CreateMenuItem(nsIMenu * aMenu, const nsString & aName, PRUint32 a
   return menuItem;
 }
 
+
 //-----------------------------------------------------
 void CreateBrowserMenus(nsIMenuBar * aMenuBar) 
 {
@@ -3325,6 +3328,7 @@ void CreateBrowserMenus(nsIMenuBar * aMenuBar)
 
 }
 
+//#endif // NS_DEBUG
 
 //-----------------------------------------------------
 nsresult
@@ -3344,6 +3348,8 @@ nsBrowserWindow::CreateMenuBar(PRInt32 aWidth)
   return NS_OK;
 }
 
+#ifdef NS_DEBUG
+
 nsEventStatus
 nsBrowserWindow::DispatchDebugMenu(PRInt32 aID)
 {