Oilpan: Move AnimationNode::EventDelegate to oilpan's heap

Source/core/animation/AnimationNodeTest.cpp

Index: Source/core/animation/AnimationNodeTest.cpp
diff --git a/Source/core/animation/AnimationNodeTest.cpp b/Source/core/animation/AnimationNodeTest.cpp
index ab626d9d7cef9f82c4b7fc33882b7180608d1321..fd0b30509d9596ec722a7aef6bc73ad39c4053a0 100644
--- a/Source/core/animation/AnimationNodeTest.cpp
+++ b/Source/core/animation/AnimationNodeTest.cpp
@@ -74,7 +74,7 @@ public:
 
     virtual void updateChildrenAndEffects() const OVERRIDE { }
     void willDetach() { }
-    TestAnimationNodeEventDelegate* eventDelegate() { return m_eventDelegate; }
+    TestAnimationNodeEventDelegate* eventDelegate() { return m_eventDelegate.get(); }
     virtual double calculateTimeToEffectChange(bool forwards, double localTime, double timeToNextIteration) const OVERRIDE
     {
         m_localTime = localTime;
@@ -95,14 +95,19 @@ public:
         return result;
     }
 
+    void trace(Visitor* visitor) OVERRIDE

[tkent, 2014/07/28 23:36:19]

This should be virtual, and call AnimationNode::trace().

[haraken, 2014/07/28 23:53:24]

Done.

+    {
+        visitor->trace(m_eventDelegate);
+    }
+
 private:
     TestAnimationNode(const Timing& specified, TestAnimationNodeEventDelegate* eventDelegate)
-        : AnimationNode(specified, adoptPtr(eventDelegate))
+        : AnimationNode(specified, adoptPtrWillBeNoop(eventDelegate))
         , m_eventDelegate(eventDelegate)
     {
     }
 
-    TestAnimationNodeEventDelegate* m_eventDelegate;
+    RawPtrWillBeMember<TestAnimationNodeEventDelegate> m_eventDelegate;
     mutable double m_localTime;
     mutable double m_timeToNextIteration;
 };