Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(480)

Unified Diff: Source/core/animation/css/CSSAnimationUpdate.h

Issue 1318543009: Oilpan: Partially ship Oilpan for core/animations (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/animation/SampledEffect.h ('k') | Source/core/animation/css/CSSAnimations.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/animation/css/CSSAnimationUpdate.h
diff --git a/Source/core/animation/css/CSSAnimationUpdate.h b/Source/core/animation/css/CSSAnimationUpdate.h
index f9669772272460e3595e6e208375a07916b68d89..af72d11631728b27e4597d2ff87e34c691530817 100644
--- a/Source/core/animation/css/CSSAnimationUpdate.h
+++ b/Source/core/animation/css/CSSAnimationUpdate.h
@@ -36,7 +36,7 @@ public:
{
}
- NewAnimation(AtomicString name, PassRefPtrWillBeRawPtr<InertEffect> effect, Timing timing, PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
+ NewAnimation(AtomicString name, InertEffect* effect, Timing timing, PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
: name(name)
, effect(effect)
, timing(timing)
@@ -52,7 +52,7 @@ public:
}
AtomicString name;
- RefPtrWillBeMember<InertEffect> effect;
+ Member<InertEffect> effect;
Timing timing;
RefPtrWillBeMember<StyleRuleKeyframes> styleRule;
unsigned styleRuleVersion;
@@ -66,7 +66,7 @@ public:
{
}
- UpdatedAnimation(AtomicString name, Animation* animation, PassRefPtrWillBeRawPtr<InertEffect> effect, Timing specifiedTiming, PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
+ UpdatedAnimation(AtomicString name, Animation* animation, InertEffect* effect, Timing specifiedTiming, PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
: name(name)
, animation(animation)
, effect(effect)
@@ -84,8 +84,8 @@ public:
}
AtomicString name;
- RawPtrWillBeMember<Animation> animation;
- RefPtrWillBeMember<InertEffect> effect;
+ Member<Animation> animation;
+ Member<InertEffect> effect;
Timing specifiedTiming;
RefPtrWillBeMember<StyleRuleKeyframes> styleRule;
unsigned styleRuleVersion;
@@ -130,8 +130,8 @@ public:
visitor->trace(snapshot);
}
- RawPtrWillBeMember<Animation> animation;
- RawPtrWillBeMember<KeyframeEffectModelBase> model;
+ Member<Animation> animation;
+ Member<KeyframeEffectModelBase> model;
CompositableStyleSnapshot snapshot;
};
@@ -141,11 +141,9 @@ public:
~CSSAnimationUpdate()
{
-#if ENABLE(OILPAN)
// For performance reasons, explicitly clear HeapVectors and
// HeapHashMaps to avoid giving a pressure on Oilpan's GC.
clear();
-#endif
}
void copy(const CSSAnimationUpdate& update)
@@ -177,7 +175,7 @@ public:
m_finishedTransitions.clear();
}
- void startAnimation(const AtomicString& animationName, PassRefPtrWillBeRawPtr<InertEffect> effect, const Timing& timing, PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
+ void startAnimation(const AtomicString& animationName, InertEffect* effect, const Timing& timing, PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
{
effect->setName(animationName);
m_newAnimations.append(NewAnimation(animationName, effect, timing, styleRule));
@@ -193,7 +191,7 @@ public:
{
m_animationsWithPauseToggled.append(name);
}
- void updateAnimation(const AtomicString& name, Animation* animation, PassRefPtrWillBeRawPtr<InertEffect> effect, const Timing& specifiedTiming,
+ void updateAnimation(const AtomicString& name, Animation* animation, InertEffect* effect, const Timing& specifiedTiming,
PassRefPtrWillBeRawPtr<StyleRuleKeyframes> styleRule)
{
m_animationsWithUpdates.append(UpdatedAnimation(name, animation, effect, specifiedTiming, styleRule));
@@ -217,7 +215,7 @@ public:
m_animationsWithStyleUpdates.append(UpdatedAnimationStyle(animation, model, snapshot));
}
- void startTransition(CSSPropertyID id, const AnimatableValue* from, const AnimatableValue* to, PassRefPtrWillBeRawPtr<InertEffect> effect)
+ void startTransition(CSSPropertyID id, const AnimatableValue* from, const AnimatableValue* to, InertEffect* effect)
{
effect->setName(getPropertyName(id));
NewTransition newTransition;
@@ -231,12 +229,12 @@ public:
void cancelTransition(CSSPropertyID id) { m_cancelledTransitions.add(id); }
void finishTransition(CSSPropertyID id) { m_finishedTransitions.add(id); }
- const WillBeHeapVector<NewAnimation>& newAnimations() const { return m_newAnimations; }
+ const HeapVector<NewAnimation>& newAnimations() const { return m_newAnimations; }
const Vector<AtomicString>& cancelledAnimationNames() const { return m_cancelledAnimationNames; }
- const WillBeHeapHashSet<RawPtrWillBeMember<const Animation>>& suppressedAnimations() const { return m_suppressedAnimations; }
+ const HeapHashSet<Member<const Animation>>& suppressedAnimations() const { return m_suppressedAnimations; }
const Vector<AtomicString>& animationsWithPauseToggled() const { return m_animationsWithPauseToggled; }
- const WillBeHeapVector<UpdatedAnimation>& animationsWithUpdates() const { return m_animationsWithUpdates; }
- const WillBeHeapVector<UpdatedAnimationStyle>& animationsWithStyleUpdates() const { return m_animationsWithStyleUpdates; }
+ const HeapVector<UpdatedAnimation>& animationsWithUpdates() const { return m_animationsWithUpdates; }
+ const HeapVector<UpdatedAnimationStyle>& animationsWithStyleUpdates() const { return m_animationsWithStyleUpdates; }
struct NewTransition {
ALLOW_ONLY_INLINE_ALLOCATION();
@@ -251,9 +249,9 @@ public:
CSSPropertyID id;
RawPtrWillBeMember<const AnimatableValue> from;
RawPtrWillBeMember<const AnimatableValue> to;
- RefPtrWillBeMember<InertEffect> effect;
+ Member<InertEffect> effect;
};
- using NewTransitionMap = WillBeHeapHashMap<CSSPropertyID, NewTransition>;
+ using NewTransitionMap = HeapHashMap<CSSPropertyID, NewTransition>;
const NewTransitionMap& newTransitions() const { return m_newTransitions; }
const HashSet<CSSPropertyID>& cancelledTransitions() const { return m_cancelledTransitions; }
const HashSet<CSSPropertyID>& finishedTransitions() const { return m_finishedTransitions; }
@@ -281,15 +279,15 @@ public:
DEFINE_INLINE_TRACE()
{
-#if ENABLE(OILPAN)
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);
visitor->trace(m_animationsWithStyleUpdates);
-#endif
}
private:
@@ -297,12 +295,12 @@ private:
// will be started. Note that there may be multiple animations present
// with the same name, due to the way in which we split up animations with
// incomplete keyframes.
- WillBeHeapVector<NewAnimation> m_newAnimations;
+ HeapVector<NewAnimation> m_newAnimations;
Vector<AtomicString> m_cancelledAnimationNames;
- WillBeHeapHashSet<RawPtrWillBeMember<const Animation>> m_suppressedAnimations;
+ HeapHashSet<Member<const Animation>> m_suppressedAnimations;
Vector<AtomicString> m_animationsWithPauseToggled;
- WillBeHeapVector<UpdatedAnimation> m_animationsWithUpdates;
- WillBeHeapVector<UpdatedAnimationStyle> m_animationsWithStyleUpdates;
+ HeapVector<UpdatedAnimation> m_animationsWithUpdates;
+ HeapVector<UpdatedAnimationStyle> m_animationsWithStyleUpdates;
NewTransitionMap m_newTransitions;
HashSet<CSSPropertyID> m_cancelledTransitions;
« no previous file with comments | « Source/core/animation/SampledEffect.h ('k') | Source/core/animation/css/CSSAnimations.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698