Title: [88063] trunk/Source/WebCore
Revision
88063
Author
[email protected]
Date
2011-06-03 14:45:05 -0700 (Fri, 03 Jun 2011)

Log Message

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:

Modified Paths

Diff

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);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to