Fix table cell background caching issue about interest rect

Source/core/paint/TableSectionPainter.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/paint/TableSectionPainter.h"
 
 #include "core/layout/LayoutTable.h"
 #include "core/layout/LayoutTableCell.h"
 #include "core/layout/LayoutTableCol.h"
 #include "core/layout/LayoutTableRow.h"
+#include "core/paint/BlockPainter.h"
 #include "core/paint/BoxClipper.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
 #include "core/paint/ObjectPainter.h"
 #include "core/paint/PaintInfo.h"
 #include "core/paint/TableCellPainter.h"
 #include "core/paint/TableRowPainter.h"
 
 namespace blink {
 
 void TableSectionPainter::paint(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
@@ skipping 131 matching lines @@
         }
     }
 }
 
 void TableSectionPainter::paintCell(LayoutTableCell* cell, const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
     LayoutPoint cellPoint = m_layoutTableSection.flipForWritingModeForChild(cell, paintOffset);
     PaintPhase paintPhase = paintInfo.phase;
     LayoutTableRow* row = toLayoutTableRow(cell->parent());
 
-    if (paintPhase == PaintPhaseBlockBackground || paintPhase == PaintPhaseChildBlockBackground) {
+    if ((paintPhase == PaintPhaseBlockBackground || paintPhase == PaintPhaseChildBlockBackground)
+        && BlockPainter(*cell).isVisibleInPaintRect(paintInfo, paintOffset)) {
         // We need to handle painting a stack of backgrounds. This stack (from bottom to top) consists of
         // the column group, column, row group, row, and then the cell.
         LayoutTableCol* column = m_layoutTableSection.table()->colElement(cell->col());
         LayoutTableCol* columnGroup = column ? column->enclosingColumnGroup() : 0;
 
         bool columnHasBackground = column && column->hasBackground();
         bool columnGroupHasBackground = columnGroup && columnGroup->hasBackground();
         bool sectionHasBackground = m_layoutTableSection.hasBackground();
         bool rowHasBackground = row->hasBackground();
 
         if (columnHasBackground || columnGroupHasBackground || sectionHasBackground || rowHasBackground) {
             TableCellPainter tableCellPainter(*cell);
-            if (!LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(*paintInfo.context, *cell, paintPhase, paintOffset)) {
-                LayoutObjectDrawingRecorder recorder(*paintInfo.context, *cell, paintPhase, tableCellPainter.paintBounds(cellPoint, TableCellPainter::AddOffsetFromParent), paintOffset);
+            if (!LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(*paintInfo.context, *cell, DisplayItem::TableCellBackgroundFromContainers, paintOffset)) {
+                LayoutObjectDrawingRecorder recorder(*paintInfo.context, *cell, DisplayItem::TableCellBackgroundFromContainers, tableCellPainter.paintBounds(cellPoint, TableCellPainter::AddOffsetFromParent), paintOffset);
                 // Column groups and columns first.
                 // FIXME: Columns and column groups do not currently support opacity, and they are being painted "too late" in
                 // the stack, since we have already opened a transparency layer (potentially) for the table row group.
                 // Note that we deliberately ignore whether or not the cell has a layer, since these backgrounds paint "behind" the
                 // cell.
                 if (columnGroupHasBackground)
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, columnGroup);
                 if (columnHasBackground)
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, column);
 
                 // Paint the row group next.
                 if (sectionHasBackground)
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, &m_layoutTableSection);
 
                 // Paint the row next, but only if it doesn't have a layer. If a row has a layer, it will be responsible for
                 // painting the row background for the cell.
                 if (rowHasBackground && !row->hasSelfPaintingLayer())
                     tableCellPainter.paintBackgroundsBehindCell(paintInfo, cellPoint, row);
             }
         }
     }
     if ((!cell->hasSelfPaintingLayer() && !row->hasSelfPaintingLayer()))
         cell->paint(paintInfo, cellPoint);
 }
 
 } // namespace blink