Fixing bug 120899. Changing the NS_WARN_IF_FALSE() macro so that it can't be mis-used in optimized builds. This change makes the macro not usable in expressions that are supposed to be evaluated in release code since the macro now expands to nothing in non-debug builds. Also eliminate NS_VALIDATE which is equally mis-usable. r=dbaron@fas.harvard.edu, sr=brendan@mozilla.org

git-svn-id: svn://10.0.0.236/trunk@112678 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/layout/generic/nsContainerFrame.cpp

@@ -431,7 +431,12 @@ nsContainerFrame::PositionFrameView(nsIPresContext* aPresContext,
         // We have the origin in the coordinate space of the containing view,
         // but we need it in the coordinate space of the parent view so do a
         // view translation
-        NS_VERIFY(TranslatePointTo(origin, containingView, parentView), "translation failed");
+#ifdef DEBUG
+        PRBool ok = 
+#endif
+        TranslatePointTo(origin, containingView, parentView);
+
+        NS_ASSERTION(ok, "translation failed");
       }
     }