SVG Web Animations: Add compositing pipeline for SVGInterpolationTypes

third_party/WebKit/Source/core/svg/SVGDocumentExtensions.h

Index: third_party/WebKit/Source/core/svg/SVGDocumentExtensions.h
diff --git a/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.h b/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.h
index f7168c6f4fb87c6ce45f8b2fb67e5206de4bd1b7..561d5ca1f34df9730e961ea4f5971c168a4f32b8 100644
--- a/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.h
+++ b/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.h
@@ -35,6 +35,7 @@ class Document;
 class LayoutSVGResourceContainer;
 class SubtreeLayoutScope;
 class SVGSVGElement;
+class SVGElement;
 class Element;
 
 class SVGDocumentExtensions : public NoBaseWillBeGarbageCollectedFinalized<SVGDocumentExtensions> {
@@ -47,6 +48,9 @@ public:
     void addTimeContainer(SVGSVGElement*);
     void removeTimeContainer(SVGSVGElement*);
 
+    // Records the SVG element as having a Web Animation on an SVG attribute that needs applying.
+    void addPendingWebAnimatedSVGElement(SVGElement&);
+
     void addResource(const AtomicString& id, LayoutSVGResourceContainer*);
     void removeResource(const AtomicString& id);
     LayoutSVGResourceContainer* resourceById(const AtomicString& id) const;
@@ -80,6 +84,8 @@ public:
 private:
     RawPtrWillBeMember<Document> m_document;
     WillBeHeapHashSet<RawPtrWillBeMember<SVGSVGElement>> m_timeContainers; // For SVG 1.2 support this will need to be made more general.
+    using SVGElementSet = WillBeHeapHashSet<RawPtrWillBeMember<SVGElement>>;

[fs, 2015/10/21 11:24:29]

In the time window between applyEffects() and serviceAnimations() - do we need
to worry about the lifetimes of the SVGElements in this set? (Once the RawPtr
part becomes irrelevant we don't - which is soon... hopefully)

[alancutter (OOO until 2018), 2015/10/26 04:13:45]

In theory SVGElements will be added and removed from m_webAnimatedSVGElements
during the call to PageAnimator::serviceScriptedAnimations() however I did not
intentionally make this a RawPtr. This is a copy+paste bug on my part from
copying m_timeContainers' type, thanks for catching it!

+    SVGElementSet m_webAnimatedSVGElements;
     HashMap<AtomicString, LayoutSVGResourceContainer*> m_resources;
     WillBeHeapHashMap<AtomicString, OwnPtrWillBeMember<SVGPendingElements>> m_pendingResources; // Resources that are pending.
     WillBeHeapHashMap<AtomicString, OwnPtrWillBeMember<SVGPendingElements>> m_pendingResourcesForRemoval; // Resources that are pending and scheduled for removal.