[webkit-reviews] review granted: [Bug 66146] Switch RenderTable* to new layout types : [Attachment 103781] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Aug 12 12:04:03 PDT 2011
Eric Seidel <eric at webkit.org> has granted Levi Weintraub <leviw at chromium.org>'s
request for review:
Bug 66146: Switch RenderTable* to new layout types
https://bugs.webkit.org/show_bug.cgi?id=66146
Attachment 103781: Patch
https://bugs.webkit.org/attachment.cgi?id=103781&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=103781&action=review
> Source/WebCore/rendering/RenderTableRow.cpp:186
> + return LayoutRect();
This function still returns an IntRect, though no?
> Source/WebCore/rendering/RenderTableSection.cpp:318
> + LayoutUnit bdesc = 0;
> + LayoutUnit ch = m_grid[r].logicalHeight.calcMinValue(0);
Such helpful variable names!
> Source/WebCore/rendering/RenderTableSection.cpp:342
> - statePusher.push(this, IntSize(x(), y()));
> + statePusher.push(this, LayoutSize(x(), y()));
I'm surprised there isn't already a location() which could be converted to a
point directly.
> Source/WebCore/rendering/RenderTableSection.cpp:432
> + LayoutUnit dh = toAdd;
Oh goodie! dh! Of course. :)
> Source/WebCore/rendering/RenderTableSection.cpp:571
> + LayoutUnit b = cell->cellBaselinePosition();
b, yes-sir. b it is!
> Source/WebCore/rendering/RenderTableSection.cpp:617
> + view()->addLayoutDelta(LayoutSize(oldCellRect.x() - cell->x(),
oldCellRect.y() - cell->y()));
rect.location - rect.location?
> Source/WebCore/rendering/RenderTableSection.cpp:631
> + LayoutSize childOffset(cell->x() - oldCellRect.x(), cell->y() -
oldCellRect.y());
cell->location() - oldcellRect.location()?
More information about the webkit-reviews
mailing list