Split SVGFilterbuilder into "builder" and "node map" parts

third_party/WebKit/Source/core/svg/graphics/filters/SVGFilterBuilder.h

 /*
  * Copyright (C) 2008 Alex Mathews <possessedpenguinbob@gmail.com>
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 13 matching lines @@
 #include "platform/graphics/filters/FilterEffect.h"
 #include "platform/heap/Handle.h"
 #include "wtf/HashMap.h"
 #include "wtf/HashSet.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/text/AtomicStringHash.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
+class FloatRect;
 class LayoutObject;
+class SVGFilterElement;
 
-class SVGFilterBuilder final : public RefCountedWillBeGarbageCollectedFinalized<SVGFilterBuilder> {
+// A map from LayoutObject -> FilterEffect and FilterEffect -> dependent (downstream) FilterEffects ("reverse DAG").
+// Used during invalidations from changes to the primitives (graph nodes).
+class SVGFilterGraphNodeMap final : public RefCountedWillBeGarbageCollectedFinalized<SVGFilterGraphNodeMap> {
 public:
+    static PassRefPtrWillBeRawPtr<SVGFilterGraphNodeMap> create()
+    {
+        return adoptRefWillBeNoop(new SVGFilterGraphNodeMap);
+    }
+
     typedef WillBeHeapHashSet<RawPtrWillBeMember<FilterEffect>> FilterEffectSet;
 
-    static PassRefPtrWillBeRawPtr<SVGFilterBuilder> create(PassRefPtrWillBeRawPtr<FilterEffect> sourceGraphic)
-    {
-        return adoptRefWillBeNoop(new SVGFilterBuilder(sourceGraphic));
-    }
-
-    void add(const AtomicString& id, PassRefPtrWillBeRawPtr<FilterEffect>);
-
-    FilterEffect* getEffectById(const AtomicString& id) const;
-    FilterEffect* lastEffect() const { return m_lastEffect.get(); }
-
-    void appendEffectToEffectReferences(PassRefPtrWillBeRawPtr<FilterEffect>, LayoutObject*);
+    void addBuiltinEffect(FilterEffect*);
+    void addPrimitive(PassRefPtrWillBeRawPtr<FilterEffect>, LayoutObject*);

[Stephen White, 2015/10/07 19:44:01]

Nit: while we're here, could we switch the order of these args, so it's key,
value?

[fs, 2015/10/08 10:53:11]

Done.

 
     inline FilterEffectSet& effectReferences(FilterEffect* effect)
     {
         // Only allowed for effects belongs to this builder.
         ASSERT(m_effectReferences.contains(effect));
         return m_effectReferences.find(effect)->value;
     }
 
     // Required to change the attributes of a filter during an svgAttributeChanged.
     inline FilterEffect* effectByRenderer(LayoutObject* object) { return m_effectRenderer.get(object); }
 
-    void clearEffects();
-    void clearResultsRecursive(FilterEffect*);
+    void invalidateDependentEffects(FilterEffect*);
 
     DECLARE_TRACE();
 
 private:
-    SVGFilterBuilder(PassRefPtrWillBeRawPtr<FilterEffect> sourceGraphic);
+    SVGFilterGraphNodeMap() = default;
 
-    inline void addBuiltinEffects()
-    {
-        for (const auto& entry : m_builtinEffects)
-            m_effectReferences.add(entry.value, FilterEffectSet());
-    }
+    // The value is a list, which contains those filter effects,
+    // which depends on the key filter effect.
+    WillBeHeapHashMap<RefPtrWillBeMember<FilterEffect>, FilterEffectSet> m_effectReferences;
+    WillBeHeapHashMap<LayoutObject*, RawPtrWillBeMember<FilterEffect>> m_effectRenderer;
+};
+
+class SVGFilterBuilder {
+    STACK_ALLOCATED();
+public:
+    SVGFilterBuilder(
+        PassRefPtrWillBeRawPtr<Filter>,
+        PassRefPtrWillBeRawPtr<FilterEffect> sourceGraphic,
+        PassRefPtrWillBeRawPtr<SVGFilterGraphNodeMap> = nullptr);
+
+    void buildGraph(SVGFilterElement&, const FloatRect&);
+
+    FilterEffect* getEffectById(const AtomicString& id) const;
+
+private:
+    void add(const AtomicString& id, PassRefPtrWillBeRawPtr<FilterEffect>);
+    void addBuiltinEffects();
 
     typedef WillBeHeapHashMap<AtomicString, RefPtrWillBeMember<FilterEffect>> NamedFilterEffectMap;
 
     NamedFilterEffectMap m_builtinEffects;
     NamedFilterEffectMap m_namedEffects;
-    // The value is a list, which contains those filter effects,
-    // which depends on the key filter effect.
-    WillBeHeapHashMap<RefPtrWillBeMember<FilterEffect>, FilterEffectSet> m_effectReferences;
-    WillBeHeapHashMap<LayoutObject*, RawPtrWillBeMember<FilterEffect>> m_effectRenderer;
 
     RefPtrWillBeMember<FilterEffect> m_lastEffect;
+    RefPtrWillBeMember<Filter> m_filter;
+    RefPtrWillBeMember<SVGFilterGraphNodeMap> m_nodeMap;
 };
 
 } // namespace blink
 
 #endif // SVGFilterBuilder_h