Move painting code for tables into *Paint classes.

Source/core/rendering/RenderTableSection.h

Index: Source/core/rendering/RenderTableSection.h
diff --git a/Source/core/rendering/RenderTableSection.h b/Source/core/rendering/RenderTableSection.h
index f9fb56d64e1e96bb807b3804701a8d59edb01016..1335b8b4b2ee14547aa08ac34250cb4287919e7b 100644
--- a/Source/core/rendering/RenderTableSection.h
+++ b/Source/core/rendering/RenderTableSection.h
@@ -224,6 +224,13 @@ public:
 
     virtual void paint(PaintInfo&, const LayoutPoint&) OVERRIDE;
 
+    // Flip the rect so it aligns with the coordinates used by the rowPos and columnPos vectors.
+    LayoutRect logicalRectForWritingModeAndDirection(const LayoutRect&) const;

[mstensho (USE GERRIT), 2014/09/19 10:50:11]

Shouldn't there be a blank line after this to make it clearer that the comment
above is only for this method?

[chrishtr, 2014/09/19 16:35:24]

Done.

+    CellSpan dirtiedRows(const LayoutRect& paintInvalidationRect) const;
+    CellSpan dirtiedColumns(const LayoutRect& paintInvalidationRect) const;
+    WillBeHeapHashSet<RawPtrWillBeMember<RenderTableCell> >& overflowingCells() { return m_overflowingCells; }
+    bool hasMultipleCellLevels() { return m_hasMultipleCellLevels; }
+
 protected:
     virtual void styleDidChange(StyleDifference, const RenderStyle* oldStyle) OVERRIDE;
     virtual bool nodeAtPoint(const HitTestRequest&, HitTestResult&, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset, HitTestAction) OVERRIDE;
@@ -240,7 +247,6 @@ private:
 
     virtual void layout() OVERRIDE;
 
-    void paintCell(RenderTableCell*, PaintInfo&, const LayoutPoint&);
     virtual void paintObject(PaintInfo&, const LayoutPoint&) OVERRIDE;
 
     virtual void imageChanged(WrappedImagePtr, const IntRect* = 0) OVERRIDE;
@@ -268,17 +274,12 @@ private:
     void updateBaselineForCell(RenderTableCell*, unsigned row, LayoutUnit& baselineDescent);
 
     bool hasOverflowingCell() const { return m_overflowingCells.size() || m_forceSlowPaintPathWithOverflowingCell; }
+
     void computeOverflowFromCells(unsigned totalRows, unsigned nEffCols);
 
     CellSpan fullTableRowSpan() const { return CellSpan(0, m_grid.size()); }
     CellSpan fullTableColumnSpan() const { return CellSpan(0, table()->columns().size()); }
 
-    // Flip the rect so it aligns with the coordinates used by the rowPos and columnPos vectors.
-    LayoutRect logicalRectForWritingModeAndDirection(const LayoutRect&) const;
-
-    CellSpan dirtiedRows(const LayoutRect& paintInvalidationRect) const;
-    CellSpan dirtiedColumns(const LayoutRect& paintInvalidationRect) const;
-
     // These two functions take a rectangle as input that has been flipped by logicalRectForWritingModeAndDirection.
     // The returned span of rows or columns is end-exclusive, and empty if start==end.
     CellSpan spannedRows(const LayoutRect& flippedRect) const;