Add a method visualRect() to DisplayItemClient.

This is to be used for computing display item bounds for raster task culling.

BUG=529938

Review URL: https://codereview.chromium.org/1517113003

Cr-Commit-Position: refs/heads/master@{#364770}

5 files changed, 13 insertions, 0 deletions

diff --git a/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp b/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
index cea4553..cde702f 100644
--- a/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
+++ b/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
@@ -158,6 +158,11 @@ GraphicsLayer::~GraphicsLayer()
     ASSERT(!m_parent);
 }
 
+IntRect GraphicsLayer::visualRect() const
+{
+    return enclosingIntRect(FloatRect(FloatPoint(0, 0), size()));
+}
+
 void GraphicsLayer::setDrawDebugRedFillForTesting(bool enabled)
 {
     s_drawDebugRedFill = enabled;
diff --git a/third_party/WebKit/Source/platform/graphics/GraphicsLayer.h b/third_party/WebKit/Source/platform/graphics/GraphicsLayer.h
index c0aaded..8fbd829 100644
--- a/third_party/WebKit/Source/platform/graphics/GraphicsLayer.h
+++ b/third_party/WebKit/Source/platform/graphics/GraphicsLayer.h
@@ -267,7 +267,9 @@ public:
     static void setDrawDebugRedFillForTesting(bool);
     ContentLayerDelegate* contentLayerDelegateForTesting() const { return m_contentLayerDelegate.get(); }
 
+    // DisplayItemClient methods
     String debugName() const final { return m_client->debugName(this); }
+    IntRect visualRect() const override;
 
 protected:
     String debugName(cc::Layer*) const;
diff --git a/third_party/WebKit/Source/platform/graphics/paint/DisplayItemClient.h b/third_party/WebKit/Source/platform/graphics/paint/DisplayItemClient.h
index eb2bc41..4ed7200 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/DisplayItemClient.h
+++ b/third_party/WebKit/Source/platform/graphics/paint/DisplayItemClient.h
@@ -6,6 +6,7 @@
 #define DisplayItemClient_h
 
 #include "platform/PlatformExport.h"
+#include "platform/geometry/IntRect.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
@@ -16,6 +17,9 @@ public:
     virtual ~DisplayItemClient() { }
 
     virtual String debugName() const = 0;
+
+    // The visual rect of this DisplayItemClient, in the space of its containing GraphicsLayer.
+    virtual IntRect visualRect() const = 0;
 };
 
 inline bool operator==(const DisplayItemClient& client1, const DisplayItemClient& client2) { return &client1 == &client2; }
diff --git a/third_party/WebKit/Source/platform/graphics/paint/PaintArtifactToSkCanvasTest.cpp b/third_party/WebKit/Source/platform/graphics/paint/PaintArtifactToSkCanvasTest.cpp
index e4753e1..47510bf 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/PaintArtifactToSkCanvasTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/paint/PaintArtifactToSkCanvasTest.cpp
@@ -68,6 +68,7 @@ public:
     SkColor color() const { return m_color; }
     PassRefPtr<SkPicture> makePicture() const { return pictureWithRect(m_rect, m_color); }
     String debugName() const final { return "<dummy>"; }
+    IntRect visualRect() const override { return IntRect(); }
 
 private:
     SkRect m_rect;
diff --git a/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp b/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
index c375021..633ad04 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.cpp
@@ -50,6 +50,7 @@ public:
     { }
 
     String debugName() const final { return m_name; }
+    IntRect visualRect() const override { return IntRect(); }
 
 private:
     String m_name;