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

Unified Diff: Source/core/animation/AnimationStack.cpp

Issue 1120003002: [Oilpan] Migrate most classes under core/animations to Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Resize expect size of Persistent Created 5 years, 7 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
Index: Source/core/animation/AnimationStack.cpp
diff --git a/Source/core/animation/AnimationStack.cpp b/Source/core/animation/AnimationStack.cpp
index d9645baebc7937b8c5962bd4550cdbd773676993..755ef82214acd567f0daf593fe5af7967591ca71 100644
--- a/Source/core/animation/AnimationStack.cpp
+++ b/Source/core/animation/AnimationStack.cpp
@@ -42,23 +42,23 @@ namespace blink {
namespace {
-void copyToActiveInterpolationMap(const WillBeHeapVector<RefPtrWillBeMember<Interpolation>>& source, ActiveInterpolationMap& target)
+void copyToActiveInterpolationMap(const HeapVector<Member<Interpolation>>& source, ActiveInterpolationMap& target)
{
for (const auto& interpolation : source) {
target.set(interpolation->property(), interpolation.get());
}
}
-bool compareEffects(const OwnPtrWillBeMember<SampledEffect>& effect1, const OwnPtrWillBeMember<SampledEffect>& effect2)
+bool compareEffects(const Member<SampledEffect>& effect1, const Member<SampledEffect>& effect2)
{
ASSERT(effect1 && effect2);
return effect1->sequenceNumber() < effect2->sequenceNumber();
}
-void copyNewAnimationsToActiveInterpolationMap(const WillBeHeapVector<RawPtrWillBeMember<InertEffect>>& newAnimations, ActiveInterpolationMap& result)
+void copyNewAnimationsToActiveInterpolationMap(const HeapVector<Member<InertEffect>>& newAnimations, ActiveInterpolationMap& result)
{
for (const auto& newAnimation : newAnimations) {
- OwnPtrWillBeRawPtr<WillBeHeapVector<RefPtrWillBeMember<Interpolation>>> sample = nullptr;
+ HeapVector<Member<Interpolation>>* sample = nullptr;
newAnimation->sample(sample);
if (sample)
copyToActiveInterpolationMap(*sample, result);
@@ -80,14 +80,14 @@ bool AnimationStack::hasActiveAnimationsOnCompositor(CSSPropertyID property) con
return false;
}
-ActiveInterpolationMap AnimationStack::activeInterpolations(AnimationStack* animationStack, const WillBeHeapVector<RawPtrWillBeMember<InertEffect>>* newAnimations, const WillBeHeapHashSet<RawPtrWillBeMember<const Animation>>* suppressedAnimations, KeyframeEffect::Priority priority, double timelineCurrentTime)
+ActiveInterpolationMap AnimationStack::activeInterpolations(AnimationStack* animationStack, const HeapVector<Member<InertEffect>>* newAnimations, const HeapHashSet<Member<const Animation>>* suppressedAnimations, KeyframeEffect::Priority priority, double timelineCurrentTime)
{
// We don't exactly know when new animations will start, but timelineCurrentTime is a good estimate.
ActiveInterpolationMap result;
if (animationStack) {
- WillBeHeapVector<OwnPtrWillBeMember<SampledEffect>>& effects = animationStack->m_effects;
+ HeapVector<Member<SampledEffect>>& effects = animationStack->m_effects;
// std::sort doesn't work with OwnPtrs
sof 2015/05/30 11:34:04 nit: s/OwnPtr/Member/ (or remove the comment, does
peria 2015/06/01 04:43:02 Done.
nonCopyingSort(effects.begin(), effects.end(), compareEffects);
animationStack->removeClearedEffects();

Powered by Google App Engine
This is Rietveld 408576698