Don't access DisplayItemClient::VisualRect() for cached display items.

third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp

Index: third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp b/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
index 6711ad15da7a65211c99ff837d61a37b846114b0..1cff8d5208adb8a037e70deeba18147794d5e841 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
@@ -30,10 +30,6 @@ class PaintControllerTestBase : public ::testing::Test {
  public:
   PaintControllerTestBase() : paint_controller_(PaintController::Create()) {}
 
-  IntRect VisualRect(const PaintArtifact& paint_artifact, size_t index) {
-    return paint_artifact.GetDisplayItemList().VisualRect(index);
-  }
-
  protected:
   PaintController& GetPaintController() { return *paint_controller_; }
 
@@ -67,7 +63,7 @@ class TestDisplayItem final : public DisplayItem {
       : DisplayItem(client, type, sizeof(*this)) {}
 
   void Replay(GraphicsContext&) const final { NOTREACHED(); }
-  void AppendToWebDisplayItemList(const IntRect&,
+  void AppendToWebDisplayItemList(const LayoutSize&,
                                   WebDisplayItemList*) const final {
     NOTREACHED();
   }
@@ -1885,18 +1881,6 @@ TEST_F(PaintControllerTestBase, SmallPaintControllerHasOnePaintChunk) {
   EXPECT_EQ(1u, paint_chunks[0].end_index);
 }
 
-TEST_F(PaintControllerTestBase, PaintArtifactWithVisualRects) {
-  FakeDisplayItemClient client("test client", LayoutRect(0, 0, 200, 100));
-
-  GraphicsContext context(GetPaintController());
-  DrawRect(context, client, kBackgroundDrawingType, FloatRect(0, 0, 100, 100));
-
-  GetPaintController().CommitNewDisplayItems(LayoutSize(20, 30));
-  const auto& paint_artifact = GetPaintController().GetPaintArtifact();
-  ASSERT_EQ(1u, paint_artifact.GetDisplayItemList().size());
-  EXPECT_EQ(IntRect(-20, -30, 200, 100), VisualRect(paint_artifact, 0));
-}
-
 void DrawPath(GraphicsContext& context,
               DisplayItemClient& client,
               DisplayItem::Type type,
@@ -1923,7 +1907,7 @@ TEST_F(PaintControllerTestBase, IsSuitableForGpuRasterizationSinglePath) {
   FakeDisplayItemClient client("test client", LayoutRect(0, 0, 200, 100));
   GraphicsContext context(GetPaintController());
   DrawPath(context, client, kBackgroundDrawingType, 1);
-  GetPaintController().CommitNewDisplayItems(LayoutSize());
+  GetPaintController().CommitNewDisplayItems();
   EXPECT_TRUE(
       GetPaintController().GetPaintArtifact().IsSuitableForGpuRasterization());
 }
@@ -1934,7 +1918,7 @@ TEST_F(PaintControllerTestBase,
   GraphicsContext context(GetPaintController());
 
   DrawPath(context, client, kBackgroundDrawingType, 50);
-  GetPaintController().CommitNewDisplayItems(LayoutSize());
+  GetPaintController().CommitNewDisplayItems();
   EXPECT_FALSE(
       GetPaintController().GetPaintArtifact().IsSuitableForGpuRasterization());
 }
@@ -1949,7 +1933,7 @@ TEST_F(PaintControllerTestBase,
     DrawPath(context, client, kBackgroundDrawingType, 50);
 
   GetPaintController().EndSkippingCache();
-  GetPaintController().CommitNewDisplayItems(LayoutSize());
+  GetPaintController().CommitNewDisplayItems();
   EXPECT_FALSE(
       GetPaintController().GetPaintArtifact().IsSuitableForGpuRasterization());
 }
@@ -1961,7 +1945,7 @@ TEST_F(PaintControllerTestBase,
   {
     GraphicsContext context(GetPaintController());
     DrawPath(context, client, kBackgroundDrawingType, 50);
-    GetPaintController().CommitNewDisplayItems(LayoutSize());
+    GetPaintController().CommitNewDisplayItems();
     EXPECT_FALSE(GetPaintController()
                      .GetPaintArtifact()
                      .IsSuitableForGpuRasterization());
@@ -1972,7 +1956,7 @@ TEST_F(PaintControllerTestBase,
   {
     GraphicsContext context(GetPaintController());
     DrawPath(context, client, kBackgroundDrawingType, 50);
-    GetPaintController().CommitNewDisplayItems(LayoutSize());
+    GetPaintController().CommitNewDisplayItems();
     EXPECT_FALSE(GetPaintController()
                      .GetPaintArtifact()
                      .IsSuitableForGpuRasterization());
@@ -1986,7 +1970,7 @@ TEST_F(PaintControllerTestBase,
   {
     GraphicsContext context(GetPaintController());
     DrawPath(context, client, kBackgroundDrawingType, 50);
-    GetPaintController().CommitNewDisplayItems(LayoutSize());
+    GetPaintController().CommitNewDisplayItems();
     EXPECT_FALSE(GetPaintController()
                      .GetPaintArtifact()
                      .IsSuitableForGpuRasterization());
@@ -1995,7 +1979,7 @@ TEST_F(PaintControllerTestBase,
   {
     GraphicsContext context(GetPaintController());
     DrawPath(context, client, kBackgroundDrawingType, 50);
-    GetPaintController().CommitNewDisplayItems(LayoutSize());
+    GetPaintController().CommitNewDisplayItems();
     EXPECT_FALSE(GetPaintController()
                      .GetPaintArtifact()
                      .IsSuitableForGpuRasterization());
@@ -2013,13 +1997,13 @@ TEST_F(
     SubsequenceRecorder subsequence_recorder(context, container);
     DrawPath(context, client, kBackgroundDrawingType, 50);
   }
-  GetPaintController().CommitNewDisplayItems(LayoutSize());
+  GetPaintController().CommitNewDisplayItems();
   EXPECT_FALSE(
       GetPaintController().GetPaintArtifact().IsSuitableForGpuRasterization());
 
   EXPECT_TRUE(
       SubsequenceRecorder::UseCachedSubsequenceIfPossible(context, container));
-  GetPaintController().CommitNewDisplayItems(LayoutSize());
+  GetPaintController().CommitNewDisplayItems();
   EXPECT_FALSE(
       GetPaintController().GetPaintArtifact().IsSuitableForGpuRasterization());
 
@@ -2050,7 +2034,7 @@ TEST_F(PaintControllerTestBase,
              FloatRect(0, 0, 100, 100));
     for (int j = 0; j < 50; ++j)
       GetPaintController().CreateAndAppend<EndClipPathDisplayItem>(client);
-    GetPaintController().CommitNewDisplayItems(LayoutSize());
+    GetPaintController().CommitNewDisplayItems();
     EXPECT_FALSE(GetPaintController()
                      .GetPaintArtifact()
                      .IsSuitableForGpuRasterization());