| Index: Source/core/animation/AnimationTimeline.cpp
|
| diff --git a/Source/core/animation/AnimationTimeline.cpp b/Source/core/animation/AnimationTimeline.cpp
|
| index 28548a928b31314deb2e781a48c3269f7d3543fb..23d7e3c71b38f8bf18eaa22969f2e9a81d1dbff4 100644
|
| --- a/Source/core/animation/AnimationTimeline.cpp
|
| +++ b/Source/core/animation/AnimationTimeline.cpp
|
| @@ -47,7 +47,7 @@ namespace blink {
|
|
|
| namespace {
|
|
|
| -bool compareAnimations(const RefPtrWillBeMember<Animation>& left, const RefPtrWillBeMember<Animation>& right)
|
| +bool compareAnimations(const Member<Animation>& left, const Member<Animation>& right)
|
| {
|
| return Animation::hasLowerPriority(left.get(), right.get());
|
| }
|
| @@ -59,12 +59,12 @@ bool compareAnimations(const RefPtrWillBeMember<Animation>& left, const RefPtrWi
|
| const double AnimationTimeline::s_minimumDelay = 0.04;
|
|
|
|
|
| -PassRefPtrWillBeRawPtr<AnimationTimeline> AnimationTimeline::create(Document* document, PassOwnPtrWillBeRawPtr<PlatformTiming> timing)
|
| +AnimationTimeline* AnimationTimeline::create(Document* document, PlatformTiming* timing)
|
| {
|
| - return adoptRefWillBeNoop(new AnimationTimeline(document, timing));
|
| + return new AnimationTimeline(document, timing);
|
| }
|
|
|
| -AnimationTimeline::AnimationTimeline(Document* document, PassOwnPtrWillBeRawPtr<PlatformTiming> timing)
|
| +AnimationTimeline::AnimationTimeline(Document* document, PlatformTiming* timing)
|
| : m_document(document)
|
| , m_zeroTime(0) // 0 is used by unit tests which cannot initialize from the loader
|
| , m_zeroTimeInitialized(false)
|
| @@ -73,7 +73,7 @@ AnimationTimeline::AnimationTimeline(Document* document, PassOwnPtrWillBeRawPtr<
|
| , m_lastCurrentTimeInternal(0)
|
| {
|
| if (!timing)
|
| - m_timing = adoptPtrWillBeNoop(new AnimationTimelineTiming(this));
|
| + m_timing = new AnimationTimelineTiming(this);
|
| else
|
| m_timing = timing;
|
|
|
| @@ -85,10 +85,6 @@ AnimationTimeline::AnimationTimeline(Document* document, PassOwnPtrWillBeRawPtr<
|
|
|
| AnimationTimeline::~AnimationTimeline()
|
| {
|
| -#if !ENABLE(OILPAN)
|
| - for (const auto& animation : m_animations)
|
| - animation->detachFromTimeline();
|
| -#endif
|
| }
|
|
|
| void AnimationTimeline::animationAttached(Animation& animation)
|
| @@ -103,7 +99,7 @@ Animation* AnimationTimeline::play(AnimationEffect* child)
|
| if (!m_document)
|
| return nullptr;
|
|
|
| - RefPtrWillBeRawPtr<Animation> animation = Animation::create(child, this);
|
| + RefPtr<Animation> animation = Animation::create(child, this);
|
| ASSERT(m_animations.contains(animation.get()));
|
|
|
| animation->play();
|
| @@ -112,9 +108,9 @@ Animation* AnimationTimeline::play(AnimationEffect* child)
|
| return animation.get();
|
| }
|
|
|
| -WillBeHeapVector<RefPtrWillBeMember<Animation>> AnimationTimeline::getAnimations()
|
| +HeapVector<Member<Animation>> AnimationTimeline::getAnimations()
|
| {
|
| - WillBeHeapVector<RefPtrWillBeMember<Animation>> animations;
|
| + HeapVector<Member<Animation>> animations;
|
| for (const auto& animation : m_animations) {
|
| if (animation->effect() && (animation->effect()->isCurrent() || animation->effect()->isInEffect()))
|
| animations.append(animation);
|
| @@ -136,10 +132,10 @@ void AnimationTimeline::serviceAnimations(TimingUpdateReason reason)
|
|
|
| m_timing->cancelWake();
|
|
|
| - WillBeHeapVector<RawPtrWillBeMember<Animation>> animations;
|
| + HeapVector<Member<Animation>> animations;
|
| animations.reserveInitialCapacity(m_animationsNeedingUpdate.size());
|
| - for (RefPtrWillBeMember<Animation> animation : m_animationsNeedingUpdate)
|
| - animations.append(animation.get());
|
| + for (Animation* animation : m_animationsNeedingUpdate)
|
| + animations.append(animation);
|
|
|
| std::sort(animations.begin(), animations.end(), Animation::hasLowerPriority);
|
|
|
| @@ -323,12 +319,10 @@ void AnimationTimeline::detachFromDocument()
|
|
|
| DEFINE_TRACE(AnimationTimeline)
|
| {
|
| -#if ENABLE(OILPAN)
|
| visitor->trace(m_document);
|
| visitor->trace(m_timing);
|
| visitor->trace(m_animationsNeedingUpdate);
|
| visitor->trace(m_animations);
|
| -#endif
|
| }
|
|
|
| } // namespace
|
|
|