Fix assertion - bug 347495 - was due to an attempt to paint children in case of invalid markup, r+sr=roc

layout/mathml/base/src/nsMathMLContainerFrame.cpp

@@ -715,17 +715,17 @@ nsMathMLContainerFrame::BuildDisplayList(nsDisplayListBuilder*   aBuilder,
                                          const nsRect&           aDirtyRect,
                                          const nsDisplayListSet& aLists)
 {
-  nsresult rv = DisplayBorderBackgroundOutline(aBuilder, aLists);
-  NS_ENSURE_SUCCESS(rv, rv);
-  
   // report an error if something wrong was found in this frame
   if (NS_MATHML_HAS_ERROR(mPresentationData.flags)) {
-    // XXX this has no visibility check ... dunno if that's right or not
+    if (!IsVisibleForPainting(aBuilder))
-    // REVIEW: That is an existing issue
+      return NS_OK;
-    rv = aLists.Content()->AppendNewToTop(new (aBuilder) nsDisplayMathMLError(this));
+
-    NS_ENSURE_SUCCESS(rv, rv);
+    return aLists.Content()->AppendNewToTop(new (aBuilder) nsDisplayMathMLError(this));
   }
 
+  nsresult rv = DisplayBorderBackgroundOutline(aBuilder, aLists);
+  NS_ENSURE_SUCCESS(rv, rv);
+
   rv = DisplayTextDecorationsAndChildren(aBuilder, aDirtyRect, aLists);
   NS_ENSURE_SUCCESS(rv, rv);