Bug 370353: Dynamically setting "visibility: collapse" to a <col> no longer works. Patch by Daniel Holbert <dholbert@mozilla.com>. r=bernd sr=dbaron a1.9=dbaron

layout/tables/nsTableFrame.cpp

@@ -1845,6 +1845,7 @@ NS_METHOD nsTableFrame::Reflow(nsPresContext*          aPresContext,
 
   PRBool haveDesiredHeight = PR_FALSE;
   PRBool reflowedChildren  = PR_FALSE;
+  SetHaveReflowedColGroups(PR_FALSE);
 
   if (aReflowState.ComputedHeight() != NS_UNCONSTRAINEDSIZE ||
       // Also check mVResize, to handle the first Reflow preceding a
@@ -3061,6 +3062,7 @@ nsTableFrame::ReflowColGroups(nsIRenderingContext *aRenderingContext)
     nsPresContext *presContext = PresContext();
     for (nsIFrame* kidFrame = mColGroups.FirstChild(); kidFrame;
          kidFrame = kidFrame->GetNextSibling()) {
+      if (NS_SUBTREE_DIRTY(kidFrame)) {
       // The column groups don't care about dimensions or reflow states.
       nsHTMLReflowState kidReflowState(presContext, kidFrame,
                                        aRenderingContext, nsSize(0,0));
@@ -3069,6 +3071,7 @@ nsTableFrame::ReflowColGroups(nsIRenderingContext *aRenderingContext)
                   cgStatus);
       FinishReflowChild(kidFrame, presContext, nsnull, kidMet, 0, 0, 0);
     }
+    }
     SetHaveReflowedColGroups(PR_TRUE);
   }
 }