[Slimming Paint] Paint the inspector overlay with GraphicsLayer DisplayList.

Source/web/PageOverlay.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  * 1. Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *
@@ skipping 14 matching lines @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "web/PageOverlay.h"
 
 #include "core/frame/Settings.h"
 #include "core/page/Page.h"
 #include "platform/graphics/GraphicsContext.h"
+#include "platform/graphics/GraphicsContextStateSaver.h"
 #include "platform/graphics/GraphicsLayer.h"
-#include "platform/graphics/GraphicsLayerClient.h"
+#include "platform/graphics/paint/DisplayItem.h"
+#include "platform/graphics/paint/DrawingRecorder.h"
 #include "public/platform/WebLayer.h"
 #include "public/web/WebPageOverlay.h"
 #include "public/web/WebViewClient.h"
 #include "web/WebViewImpl.h"
 
 namespace blink {
 
-namespace {
-
-WebCanvas* ToWebCanvas(GraphicsContext* gc)
-{
-    return gc->canvas();
-}
-
-} // namespace
-
 PassOwnPtr<PageOverlay> PageOverlay::create(WebViewImpl* viewImpl, WebPageOverlay* overlay)
 {
     return adoptPtr(new PageOverlay(viewImpl, overlay));
 }
 
+PassOwnPtr<PageOverlay> PageOverlay::create(WebViewImpl* viewImpl, PageOverlay::Painter* overlayPainter)
+{
+    return adoptPtr(new PageOverlay(viewImpl, overlayPainter));
+}
+
 PageOverlay::PageOverlay(WebViewImpl* viewImpl, WebPageOverlay* overlay)
     : m_viewImpl(viewImpl)
     , m_overlay(overlay)
+    , m_overlayPainter(nullptr)
     , m_zOrder(0)
 {
 }
 
-class OverlayGraphicsLayerClientImpl : public GraphicsLayerClient {
-public:
-    static PassOwnPtr<OverlayGraphicsLayerClientImpl> create(WebPageOverlay* overlay)
-    {
-        return adoptPtr(new OverlayGraphicsLayerClientImpl(overlay));
-    }
-
-    virtual ~OverlayGraphicsLayerClientImpl() { }
-
-    virtual void paintContents(const GraphicsLayer*, GraphicsContext& gc, GraphicsLayerPaintingPhase, const IntRect& inClip)
-    {
-        gc.save();
-        m_overlay->paintPageOverlay(ToWebCanvas(&gc));
-        gc.restore();
-    }
-
-    virtual String debugName(const GraphicsLayer* graphicsLayer) override
-    {
-        return String("WebViewImpl Page Overlay Content Layer");
-    }
-
-private:
-    explicit OverlayGraphicsLayerClientImpl(WebPageOverlay* overlay)
-        : m_overlay(overlay)
-    {
-    }
-
-    WebPageOverlay* m_overlay;
-};
+PageOverlay::PageOverlay(WebViewImpl* viewImpl, PageOverlay::Painter* overlayPainter)
+    : m_viewImpl(viewImpl)
+    , m_overlay(nullptr)
+    , m_overlayPainter(overlayPainter)
+    , m_zOrder(0)
+{
+}
 
 void PageOverlay::clear()
 {
     invalidateWebFrame();
 
     if (m_layer) {
         m_layer->removeFromParent();
         if (Page* page = m_viewImpl->page())
             page->inspectorController().didRemovePageOverlay(m_layer.get());
         m_layer = nullptr;
-        m_layerClient = nullptr;
     }
 }
 
 void PageOverlay::update()
 {
     invalidateWebFrame();
 
     if (!m_layer) {
-        m_layerClient = OverlayGraphicsLayerClientImpl::create(m_overlay);
-        m_layer = GraphicsLayer::create(m_viewImpl->graphicsLayerFactory(), m_layerClient.get());
+        m_layer = GraphicsLayer::create(m_viewImpl->graphicsLayerFactory(), this /* client */);
         m_layer->setDrawsContent(true);
 
         if (Page* page = m_viewImpl->page())
             page->inspectorController().willAddPageOverlay(m_layer.get());
 
         // This is required for contents of overlay to stay in sync with the page while scrolling.
         WebLayer* platformLayer = m_layer->platformLayer();
         platformLayer->setShouldScrollOnMainThread(true);
     }
 
     FloatSize size(m_viewImpl->size());
     if (size != m_layer->size()) {
         // Triggers re-adding to root layer to ensure that we are on top of
         // scrollbars.
         m_layer->removeFromParent();
         m_layer->setSize(size);
     }
 
     m_viewImpl->setOverlayLayer(m_layer.get());
     m_layer->setNeedsDisplay();
 }
 
 void PageOverlay::paintWebFrame(GraphicsContext& gc)
 {
-    if (!m_viewImpl->isAcceleratedCompositingActive()) {
-        gc.save();
-        m_overlay->paintPageOverlay(ToWebCanvas(&gc));
-        gc.restore();
+    if (m_overlay) {
+        // We clip the painted contents to the WebViewImpl, because
+        // DrawingRecorder expects to know bounds.
+        DisplayItemClient displayItemClient = static_cast<DisplayItemClient>(static_cast<void*>(this));
+        IntSize size = m_viewImpl->size();
+        FloatRect clipRect(FloatPoint(0, 0), size);
+        DrawingRecorder recorder(&gc, displayItemClient, DisplayItem::PageOverlay, clipRect);
+        GraphicsContextStateSaver saver(gc);
+        gc.clipRect(clipRect);
+        m_overlay->paintPageOverlay(gc.canvas());
+    } else if (m_overlayPainter) {
+        // FIXME: Maybe we need to make the callee responsible for saving.
+        // We can't save a GraphicsContext unless we have a canvas.
+        m_overlayPainter->paintPageOverlay(gc);
     }
 }
 
+void PageOverlay::paintContents(const GraphicsLayer*, GraphicsContext& gc, GraphicsLayerPaintingPhase, const IntRect& inClip)
+{
+    paintWebFrame(gc);
+}
+
+String PageOverlay::debugName(const GraphicsLayer*)
+{
+    return "WebViewImpl Page Overlay Content Layer";
+}
+
 void PageOverlay::invalidateWebFrame()
 {
     // WebPageOverlay does the actual painting of the overlay.
     // Here we just make sure to invalidate.
     if (!m_viewImpl->isAcceleratedCompositingActive()) {
         // FIXME: able to invalidate a smaller rect.
         // FIXME: Is it important to just invalidate a smaller rect given that
         // this is not on a critical codepath? In order to do so, we'd
         // have to take scrolling into account.
         const WebSize& size = m_viewImpl->size();
         WebRect damagedRect(0, 0, size.width, size.height);
         if (m_viewImpl->client())
             m_viewImpl->client()->didInvalidateRect(damagedRect);
     }
 }
 
 } // namespace blink