[Slimming Paint] Implement deferred SVG filters

Source/core/rendering/svg/SVGRenderingContext.cpp

 /*
  * Copyright (C) 2007, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Google, Inc.  All rights reserved.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 16 matching lines @@
 
 #include "core/frame/FrameHost.h"
 #include "core/rendering/PaintInfo.h"
 #include "core/rendering/RenderLayer.h"
 #include "core/rendering/svg/RenderSVGResourceFilter.h"
 #include "core/rendering/svg/RenderSVGResourceMasker.h"
 #include "core/rendering/svg/SVGRenderSupport.h"
 #include "core/rendering/svg/SVGResources.h"
 #include "core/rendering/svg/SVGResourcesCache.h"
 #include "platform/FloatConversion.h"
+#include "platform/graphics/paint/DrawingRecorder.h"
+#include "third_party/skia/include/core/SkPicture.h"
 
 namespace blink {
 
 SVGRenderingContext::~SVGRenderingContext()
 {
     if (m_filter) {
         ASSERT(SVGResourcesCache::cachedResourcesForRenderObject(m_object));
         ASSERT(SVGResourcesCache::cachedResourcesForRenderObject(m_object)->filter() == m_filter);
-        m_filter->finishEffect(m_object, m_paintInfo.context);
+        DrawingRecorder recorder(m_originalPaintInfo->context, m_object->displayItemClient(), DisplayItem::SVGFilter, LayoutRect::infiniteRect());
+        m_filter->finishEffect(m_object, m_originalPaintInfo->context, m_paintInfo.context);
+
+        // Reset the main context after the filter effect has been completed.
+        m_paintInfo.context = m_originalPaintInfo->context;
     }
 
     if (m_masker) {
         ASSERT(SVGResourcesCache::cachedResourcesForRenderObject(m_object));
         ASSERT(SVGResourcesCache::cachedResourcesForRenderObject(m_object)->masker() == m_masker);
         m_masker->finishEffect(m_object, m_paintInfo.context);
     }
 
     if (m_clipper) {
         ASSERT(SVGResourcesCache::cachedResourcesForRenderObject(m_object));
@@ skipping 86 matching lines @@
     }
     return true;
 }
 
 bool SVGRenderingContext::applyFilterIfNecessary(SVGResources* resources)
 {
     if (!resources) {
         if (m_object->style()->svgStyle().hasFilter())
             return false;
     } else if (RenderSVGResourceFilter* filter = resources->filter()) {
-        if (!filter->prepareEffect(m_object, m_paintInfo.context))
+        if (RuntimeEnabledFeatures::slimmingPaintEnabled()) {
+            m_filterContentDisplayItemList = DisplayItemList::create();
+            m_filterContentGraphicsContext = adoptPtr(new GraphicsContext(nullptr, m_filterContentDisplayItemList.get()));
+        }
+
+        // Instead of drawing into the regular context, we draw into a (sometimes separate)
+        // context just for the contents of the filter.
+        GraphicsContext* filterContents = m_filterContentGraphicsContext ? m_filterContentGraphicsContext.get() : m_paintInfo.context;

[chrishtr, 2015/01/24 00:02:02]

Declare filterContents before line 158 and init to m_paintInfo.context, then
override it after line 160. This avoids two conditionals that always trigger on
the same conditions.

Also, do you mean filterContext, not filterContents?

[pdr., 2015/01/24 01:13:36]

Done. Refactored this even further and removed the filterConte(nt/xt).

+        if (!filter->prepareEffect(m_object, filterContents))
             return false;
+
         m_filter = filter;
 
+        if (m_filterContentGraphicsContext)
+            m_paintInfo.context = m_filterContentGraphicsContext.get();
+
         // Because we cache the filter contents and do not invalidate on paint
         // invalidation rect changes, we need to paint the entire filter region
         // so elements outside the initial paint (due to scrolling, etc) paint.
         m_paintInfo.rect = LayoutRect::infiniteIntRect();
     }
     return true;
 }
 
 bool SVGRenderingContext::isIsolationInstalled() const
 {
@@ skipping 70 matching lines @@
 {
     ASSERT(context);
     ASSERT(item);
     ASSERT(!item->needsLayout());
 
     PaintInfo info(context, LayoutRect::infiniteIntRect(), PaintPhaseForeground, PaintBehaviorNormal);
     item->paint(info, IntPoint());
 }
 
 } // namespace blink