Modified: trunk/Source/WebCore/ChangeLog (88062 => 88063)
--- trunk/Source/WebCore/ChangeLog 2011-06-03 21:39:13 UTC (rev 88062)
+++ trunk/Source/WebCore/ChangeLog 2011-06-03 21:45:05 UTC (rev 88063)
@@ -1,3 +1,20 @@
+2011-06-03 Levi Weintraub <[email protected]>
+
+ Reviewed by Eric Seidel.
+
+ Switch paintCell to use IntPoint
+ https://bugs.webkit.org/show_bug.cgi?id=62033
+
+ Switching paintCell to take an IntPoint representing the paint offset
+ instead of a pair of ints.
+
+ No new tests since this is simple refactoring.
+
+ * rendering/RenderTableSection.cpp:
+ (WebCore::RenderTableSection::paintCell):
+ (WebCore::RenderTableSection::paintObject):
+ * rendering/RenderTableSection.h:
+
2011-06-03 Martin Robinson <[email protected]>
Reviewed by Dimitri Glazkov.
Modified: trunk/Source/WebCore/rendering/RenderTableSection.cpp (88062 => 88063)
--- trunk/Source/WebCore/rendering/RenderTableSection.cpp 2011-06-03 21:39:13 UTC (rev 88062)
+++ trunk/Source/WebCore/rendering/RenderTableSection.cpp 2011-06-03 21:45:05 UTC (rev 88063)
@@ -914,9 +914,9 @@
return elem1->row() < elem2->row();
}
-void RenderTableSection::paintCell(RenderTableCell* cell, PaintInfo& paintInfo, int tx, int ty)
+void RenderTableSection::paintCell(RenderTableCell* cell, PaintInfo& paintInfo, const IntPoint& paintOffset)
{
- IntPoint cellPoint = flipForWritingMode(cell, IntPoint(tx, ty), ParentToChildFlippingAdjustment);
+ IntPoint cellPoint = flipForWritingMode(cell, paintOffset, ParentToChildFlippingAdjustment);
PaintPhase paintPhase = paintInfo.phase;
RenderTableRow* row = toRenderTableRow(cell->parent());
@@ -1018,7 +1018,7 @@
RenderTableCell* cell = current.primaryCell();
if (!cell || (r > startrow && primaryCellAt(r - 1, c) == cell) || (c > startcol && primaryCellAt(r, c - 1) == cell))
continue;
- paintCell(cell, paintInfo, tx, ty);
+ paintCell(cell, paintInfo, IntPoint(tx, ty));
}
}
} else {
@@ -1045,7 +1045,7 @@
int size = cells.size();
// Paint the cells.
for (int i = 0; i < size; ++i)
- paintCell(cells[i], paintInfo, tx, ty);
+ paintCell(cells[i], paintInfo, IntPoint(tx, ty));
}
}
}
Modified: trunk/Source/WebCore/rendering/RenderTableSection.h (88062 => 88063)
--- trunk/Source/WebCore/rendering/RenderTableSection.h 2011-06-03 21:39:13 UTC (rev 88062)
+++ trunk/Source/WebCore/rendering/RenderTableSection.h 2011-06-03 21:45:05 UTC (rev 88063)
@@ -133,7 +133,7 @@
virtual void removeChild(RenderObject* oldChild);
virtual void paint(PaintInfo&, int tx, int ty);
- virtual void paintCell(RenderTableCell*, PaintInfo&, int tx, int ty);
+ virtual void paintCell(RenderTableCell*, PaintInfo&, const IntPoint&);
virtual void paintObject(PaintInfo&, int tx, int ty);
virtual void imageChanged(WrappedImagePtr, const IntRect* = 0);