Fix table cell background caching issue about interest rect

Source/core/paint/DisplayItemListPaintTest.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 "platform/graphics/paint/DisplayItemList.h"
 
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutText.h"
 #include "core/layout/LayoutView.h"
@@ skipping 320 matching lines @@
         TestDisplayItem(container1, DisplayItem::BoxDecorationBackground),
         TestDisplayItem(content1, DisplayItem::BoxDecorationBackground),
         TestDisplayItem(container1, DisplayItem::EndSubsequence),
         TestDisplayItem(container2, DisplayItem::BeginSubsequence),
         TestDisplayItem(container2, DisplayItem::BoxDecorationBackground),
         TestDisplayItem(content2, DisplayItem::BoxDecorationBackground),
         TestDisplayItem(container2, DisplayItem::EndSubsequence),
         TestDisplayItem(html, DisplayItem::EndSubsequence));
 }
 
+// TODO(wangxianzhu): Create a version for slimming paint v2 when it supports interest rect
+TEST_F(DisplayItemListPaintTest, TableCellBackgroundInterestRect)

[chrishtr, 2015/09/03 17:25:34]

Put this in a new TableCellTest?

[Xianzhu, 2015/09/04 16:52:56]

Done.

+{
+    setBodyInnerHTML(
+        "<style>"
+        "  td { width: 200px; height: 200px; border: none; }"
+        "  tr { background-color: blue; }"
+        "  table { border: none; border-spacing: 0; border-collapse: collapse; }"
+        "</style>"
+        "<table>"
+        "  <tr><td id='cell1'></td></tr>"
+        "  <tr><td id='cell2'></td></tr>"
+        "</table>");
+
+    LayoutView& layoutView = *document().layoutView();
+    DeprecatedPaintLayer& rootLayer = *layoutView.layer();
+    LayoutObject& cell1 = *document().getElementById("cell1")->layoutObject();
+    LayoutObject& cell2 = *document().getElementById("cell2")->layoutObject();
+
+    GraphicsContext context(&rootDisplayItemList());
+    DeprecatedPaintLayerPaintingInfo paintingInfo(&rootLayer, LayoutRect(0, 0, 200, 200), GlobalPaintNormalPhase, LayoutSize());
+    DeprecatedPaintLayerPainter(rootLayer).paintLayerContents(&context, paintingInfo, PaintLayerPaintingCompositingAllPhases);
+    rootDisplayItemList().commitNewDisplayItems();
+
+    EXPECT_DISPLAY_LIST_BASE(rootDisplayItemList().displayItems(), 2,
+        TestDisplayItem(layoutView, DisplayItem::BoxDecorationBackground),
+        TestDisplayItem(cell1, DisplayItem::TableCellBackgroundFromContainers));
+
+    DeprecatedPaintLayerPaintingInfo paintingInfo1(&rootLayer, LayoutRect(0, 300, 200, 200), GlobalPaintNormalPhase, LayoutSize());
+    DeprecatedPaintLayerPainter(rootLayer).paintLayerContents(&context, paintingInfo1, PaintLayerPaintingCompositingAllPhases);
+    rootDisplayItemList().commitNewDisplayItems();
+
+    EXPECT_DISPLAY_LIST_BASE(rootDisplayItemList().displayItems(), 2,
+        TestDisplayItem(layoutView, DisplayItem::BoxDecorationBackground),
+        TestDisplayItem(cell2, DisplayItem::TableCellBackgroundFromContainers));
+}
+
 } // namespace blink