Oilpan: Unship oilpan from temporary animation objects

Source/core/animation/css/CSSAnimationUpdate.h

Index: Source/core/animation/css/CSSAnimationUpdate.h
diff --git a/Source/core/animation/css/CSSAnimationUpdate.h b/Source/core/animation/css/CSSAnimationUpdate.h
index af72d11631728b27e4597d2ff87e34c691530817..a51bca3ec9408ba21f45b5fac078fb8a568fa8b9 100644
--- a/Source/core/animation/css/CSSAnimationUpdate.h
+++ b/Source/core/animation/css/CSSAnimationUpdate.h
@@ -95,21 +95,11 @@ public:
         ALLOW_ONLY_INLINE_ALLOCATION();
     public:
         struct CompositableStyleSnapshot {
-            DISALLOW_ALLOCATION();

[sof, 2015/09/02 13:02:12]

Won't do any harm to keep?

-
         public:
-            RefPtrWillBeMember<AnimatableValue> opacity;
-            RefPtrWillBeMember<AnimatableValue> transform;
-            RefPtrWillBeMember<AnimatableValue> webkitFilter;
-            RefPtrWillBeMember<AnimatableValue> backdropFilter;
-
-            DEFINE_INLINE_TRACE()
-            {
-                visitor->trace(opacity);
-                visitor->trace(transform);
-                visitor->trace(webkitFilter);
-                visitor->trace(backdropFilter);
-            }
+            RefPtr<AnimatableValue> opacity;
+            RefPtr<AnimatableValue> transform;
+            RefPtr<AnimatableValue> webkitFilter;
+            RefPtr<AnimatableValue> backdropFilter;
         };
 
         UpdatedAnimationStyle()
@@ -127,7 +117,6 @@ public:
         {
             visitor->trace(animation);
             visitor->trace(model);
-            visitor->trace(snapshot);
         }
 
         Member<Animation> animation;
@@ -241,14 +230,12 @@ public:
     public:
         DEFINE_INLINE_TRACE()
         {
-            visitor->trace(from);
-            visitor->trace(to);
             visitor->trace(effect);
         }
 
         CSSPropertyID id;
-        RawPtrWillBeMember<const AnimatableValue> from;
-        RawPtrWillBeMember<const AnimatableValue> to;
+        const AnimatableValue* from;
+        const AnimatableValue* to;
         Member<InertEffect> effect;
     };
     using NewTransitionMap = HeapHashMap<CSSPropertyID, NewTransition>;
@@ -280,10 +267,6 @@ public:
     DEFINE_INLINE_TRACE()
     {
         visitor->trace(m_newTransitions);
-#if ENABLE(OILPAN)
-        visitor->trace(m_activeInterpolationsForAnimations);
-        visitor->trace(m_activeInterpolationsForTransitions);
-#endif
         visitor->trace(m_newAnimations);
         visitor->trace(m_suppressedAnimations);
         visitor->trace(m_animationsWithUpdates);
@@ -308,6 +291,8 @@ private:
 
     ActiveInterpolationMap m_activeInterpolationsForAnimations;
     ActiveInterpolationMap m_activeInterpolationsForTransitions;
+
+    friend class PendingAnimationUpdate;
 };
 
 } // namespace blink