| Index: Source/core/rendering/RenderTableSection.cpp
|
| diff --git a/Source/core/rendering/RenderTableSection.cpp b/Source/core/rendering/RenderTableSection.cpp
|
| index 9504b4c4817b3a21fbd813fb02db63002a579705..a383752c2e91f841ca76d73c05dd8729b7c0833a 100644
|
| --- a/Source/core/rendering/RenderTableSection.cpp
|
| +++ b/Source/core/rendering/RenderTableSection.cpp
|
| @@ -41,7 +41,7 @@ namespace blink {
|
|
|
| using namespace HTMLNames;
|
|
|
| -// Those 2 variables are used to balance the memory consumption vs the repaint time on big tables.
|
| +// Those 2 variables are used to balance the memory consumption vs the paint invalidation time on big tables.
|
| static unsigned gMinTableSizeToUseFastPaintPathWithOverflowingCell = 75 * 75;
|
| static float gMaxAllowedOverflowingCellRatioForFastPaintPath = 0.1f;
|
|
|
| @@ -1013,9 +1013,9 @@ void RenderTableSection::layoutRows()
|
|
|
| LayoutSize childOffset(cell->location() - oldCellRect.location());
|
| if (childOffset.width() || childOffset.height()) {
|
| - // If the child moved, we have to repaint it as well as any floating/positioned
|
| + // If the child moved, we have to issue paint invalidations to it as well as any floating/positioned
|
| // descendants. An exception is if we need a layout. In this case, we know we're going to
|
| - // repaint ourselves (and the child) anyway.
|
| + // issue paint invalidations ourselves (and the child) anyway.
|
| if (!table()->selfNeedsLayout() && cell->checkForPaintInvalidation())
|
| cell->setMayNeedPaintInvalidation(true);
|
| }
|
| @@ -1250,7 +1250,7 @@ static inline bool compareCellPositions(RenderTableCell* elem1, RenderTableCell*
|
| }
|
|
|
| // This comparison is used only when we have overflowing cells as we have an unsorted array to sort. We thus need
|
| -// to sort both on rows and columns to properly repaint.
|
| +// to sort both on rows and columns to properly issue paint invalidations.
|
| static inline bool compareCellPositionsWithOverflowingCells(RenderTableCell* elem1, RenderTableCell* elem2)
|
| {
|
| if (elem1->rowIndex() != elem2->rowIndex())
|
| @@ -1315,7 +1315,7 @@ CellSpan RenderTableSection::dirtiedRows(const LayoutRect& damageRect) const
|
|
|
| CellSpan coveredRows = spannedRows(damageRect);
|
|
|
| - // To repaint the border we might need to repaint first or last row even if they are not spanned themselves.
|
| + // To issue paint invalidations for the border we might need to paint invalidate the first or last row even if they are not spanned themselves.
|
| if (coveredRows.start() >= m_rowPos.size() - 1 && m_rowPos[m_rowPos.size() - 1] + table()->outerBorderAfter() >= damageRect.y())
|
| --coveredRows.start();
|
|
|
| @@ -1333,7 +1333,7 @@ CellSpan RenderTableSection::dirtiedColumns(const LayoutRect& damageRect) const
|
| CellSpan coveredColumns = spannedColumns(damageRect);
|
|
|
| const Vector<int>& columnPos = table()->columnPositions();
|
| - // To repaint the border we might need to repaint first or last column even if they are not spanned themselves.
|
| + // To issue paint invalidations for the border we might need to paint invalidate the first or last column even if they are not spanned themselves.
|
| if (coveredColumns.start() >= columnPos.size() - 1 && columnPos[columnPos.size() - 1] + table()->outerBorderEnd() >= damageRect.x())
|
| --coveredColumns.start();
|
|
|
| @@ -1398,10 +1398,10 @@ CellSpan RenderTableSection::spannedColumns(const LayoutRect& flippedRect) const
|
|
|
| void RenderTableSection::paintObject(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
|
| {
|
| - LayoutRect localRepaintRect = paintInfo.rect;
|
| - localRepaintRect.moveBy(-paintOffset);
|
| + LayoutRect localPaintInvalidationRect = paintInfo.rect;
|
| + localPaintInvalidationRect.moveBy(-paintOffset);
|
|
|
| - LayoutRect tableAlignedRect = logicalRectForWritingModeAndDirection(localRepaintRect);
|
| + LayoutRect tableAlignedRect = logicalRectForWritingModeAndDirection(localPaintInvalidationRect);
|
|
|
| CellSpan dirtiedRows = this->dirtiedRows(tableAlignedRect);
|
| CellSpan dirtiedColumns = this->dirtiedColumns(tableAlignedRect);
|
| @@ -1446,7 +1446,7 @@ void RenderTableSection::paintObject(PaintInfo& paintInfo, const LayoutPoint& pa
|
| ASSERT(m_overflowingCells.size() < totalRows * totalCols * gMaxAllowedOverflowingCellRatioForFastPaintPath);
|
| #endif
|
|
|
| - // To make sure we properly repaint the section, we repaint all the overflowing cells that we collected.
|
| + // To make sure we properly paint invalidate the section, we paint invalidated all the overflowing cells that we collected.
|
| Vector<RenderTableCell*> cells;
|
| copyToVector(m_overflowingCells, cells);
|
|
|
| @@ -1495,7 +1495,7 @@ void RenderTableSection::paintObject(PaintInfo& paintInfo, const LayoutPoint& pa
|
|
|
| void RenderTableSection::imageChanged(WrappedImagePtr, const IntRect*)
|
| {
|
| - // FIXME: Examine cells and repaint only the rect the image paints in.
|
| + // FIXME: Examine cells and issue paint invalidations of only the rect the image paints in.
|
| paintInvalidationForWholeRenderer();
|
| }
|
|
|
|
|