Changed table frame code to call DeleteFrameList() so that the col group

frames are deleted

layout/html/table/src/nsTableFrame.cpp

@@ -307,6 +307,13 @@ nsTableFrame::~nsTableFrame()
     delete mColCache;
 }
 
+NS_IMETHODIMP
+nsTableFrame::DeleteFrame(nsIPresContext& aPresContext)
+{
+  DeleteFrameList(aPresContext, &mColGroups);
+  return nsHTMLContainerFrame::DeleteFrame(aPresContext);
+}
+
 NS_IMETHODIMP
 nsTableFrame::SetInitialChildList(nsIPresContext& aPresContext,
                                   nsIAtom*        aListName,

layout/html/table/src/nsTableFrame.h

@@ -49,9 +49,12 @@ struct nsStyleSpacing;
 /* ============================================================================ */
 
 /** nsTableFrame maps the inner portion of a table (everything except captions.)
-  * Used as a pseudo-frame within nsTableOuterFrame, 
-  * it may also be used stand-alone as the top-level frame.
-  * The meaningful child frames of nsTableFrame map rowgroups.
+  * Used as a pseudo-frame within nsTableOuterFrame, it may also be used
+  * stand-alone as the top-level frame.
+  *
+  * The flowed child list contains row group framess. There is also an additional
+  * named child list:
+  * - "ColGroup-list" which contains the col group frames
   *
   * @author  sclark
   *
@@ -77,6 +80,8 @@ public:
   // nsISupports
   NS_IMETHOD QueryInterface(const nsIID& aIID, void** aInstancePtr);
 
+  NS_IMETHOD DeleteFrame(nsIPresContext& aPresContext);
+
   /** helper method for determining if this is a nested table or not */
   PRBool IsNested(const nsHTMLReflowState& aReflowState, const nsStylePosition *& aPosition) const;
 

layout/tables/nsTableFrame.cpp

@@ -307,6 +307,13 @@ nsTableFrame::~nsTableFrame()
     delete mColCache;
 }
 
+NS_IMETHODIMP
+nsTableFrame::DeleteFrame(nsIPresContext& aPresContext)
+{
+  DeleteFrameList(aPresContext, &mColGroups);
+  return nsHTMLContainerFrame::DeleteFrame(aPresContext);
+}
+
 NS_IMETHODIMP
 nsTableFrame::SetInitialChildList(nsIPresContext& aPresContext,
                                   nsIAtom*        aListName,

layout/tables/nsTableFrame.h

@@ -49,9 +49,12 @@ struct nsStyleSpacing;
 /* ============================================================================ */
 
 /** nsTableFrame maps the inner portion of a table (everything except captions.)
-  * Used as a pseudo-frame within nsTableOuterFrame, 
-  * it may also be used stand-alone as the top-level frame.
-  * The meaningful child frames of nsTableFrame map rowgroups.
+  * Used as a pseudo-frame within nsTableOuterFrame, it may also be used
+  * stand-alone as the top-level frame.
+  *
+  * The flowed child list contains row group framess. There is also an additional
+  * named child list:
+  * - "ColGroup-list" which contains the col group frames
   *
   * @author  sclark
   *
@@ -77,6 +80,8 @@ public:
   // nsISupports
   NS_IMETHOD QueryInterface(const nsIID& aIID, void** aInstancePtr);
 
+  NS_IMETHOD DeleteFrame(nsIPresContext& aPresContext);
+
   /** helper method for determining if this is a nested table or not */
   PRBool IsNested(const nsHTMLReflowState& aReflowState, const nsStylePosition *& aPosition) const;