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

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

Issue 225073004: Oilpan: Completely move core/animations/ to oilpan's heap (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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/AnimationPlayerTest.cpp
diff --git a/Source/core/animation/AnimationPlayerTest.cpp b/Source/core/animation/AnimationPlayerTest.cpp
index 842cccb9952887aed486a0f0b1b63d5ceba00d31..c666a55f93a25eb91ef49b9a540286689ba2b91f 100644
--- a/Source/core/animation/AnimationPlayerTest.cpp
+++ b/Source/core/animation/AnimationPlayerTest.cpp
@@ -68,7 +68,7 @@ protected:
updateTimeline(0);
}
- PassRefPtr<Animation> makeAnimation(double duration = 30, double playbackRate = 1)
+ PassRefPtrWillBeRawPtr<Animation> makeAnimation(double duration = 30, double playbackRate = 1)
{
Timing timing;
timing.iterationDuration = duration;
@@ -84,8 +84,8 @@ protected:
}
RefPtr<Document> document;
- RefPtr<DocumentTimeline> timeline;
- RefPtr<AnimationPlayer> player;
+ RefPtrWillBePersistent<DocumentTimeline> timeline;
+ RefPtrWillBePersistent<AnimationPlayer> player;
TrackExceptionState exceptionState;
};
@@ -562,8 +562,8 @@ TEST_F(AnimationAnimationPlayerTest, SetSource)
{
player = timeline->createAnimationPlayer(0);
player->setStartTimeInternal(0);
- RefPtr<TimedItem> source1 = makeAnimation();
- RefPtr<TimedItem> source2 = makeAnimation();
+ RefPtrWillBeRawPtr<TimedItem> source1 = makeAnimation();
+ RefPtrWillBeRawPtr<TimedItem> source2 = makeAnimation();
player->setSource(source1.get());
EXPECT_EQ(source1, player->source());
EXPECT_EQ(0, player->currentTimeInternal());
@@ -621,7 +621,7 @@ TEST_F(AnimationAnimationPlayerTest, AnimationPlayersReturnTimeToNextEffect)
timing.startDelay = 1;
timing.iterationDuration = 1;
timing.endDelay = 1;
- RefPtr<Animation> animation = Animation::create(0, nullptr, timing);
+ RefPtrWillBeRawPtr<Animation> animation = Animation::create(0, nullptr, timing);
player = timeline->createAnimationPlayer(animation.get());
player->setStartTimeInternal(0);
@@ -698,8 +698,8 @@ TEST_F(AnimationAnimationPlayerTest, AttachedAnimationPlayers)
RefPtrWillBeRawPtr<Element> element = document->createElement("foo", ASSERT_NO_EXCEPTION);
Timing timing;
- RefPtr<Animation> animation = Animation::create(element.get(), nullptr, timing);
- RefPtr<AnimationPlayer> player = timeline->createAnimationPlayer(animation.get());
+ RefPtrWillBeRawPtr<Animation> animation = Animation::create(element.get(), nullptr, timing);
+ RefPtrWillBeRawPtr<AnimationPlayer> player = timeline->createAnimationPlayer(animation.get());
player->setStartTime(0);
timeline->serviceAnimations(TimingUpdateForAnimationFrame);
EXPECT_EQ(1, element->activeAnimations()->players().find(player.get())->value);
@@ -712,17 +712,17 @@ TEST_F(AnimationAnimationPlayerTest, HasLowerPriority)
{
// Sort time defaults to timeline current time
updateTimeline(15);
- RefPtr<AnimationPlayer> player1 = timeline->createAnimationPlayer(0);
- RefPtr<AnimationPlayer> player2 = timeline->createAnimationPlayer(0);
+ RefPtrWillBeRawPtr<AnimationPlayer> player1 = timeline->createAnimationPlayer(0);
+ RefPtrWillBeRawPtr<AnimationPlayer> player2 = timeline->createAnimationPlayer(0);
player2->setStartTimeInternal(10);
- RefPtr<AnimationPlayer> player3 = timeline->createAnimationPlayer(0);
- RefPtr<AnimationPlayer> player4 = timeline->createAnimationPlayer(0);
+ RefPtrWillBeRawPtr<AnimationPlayer> player3 = timeline->createAnimationPlayer(0);
+ RefPtrWillBeRawPtr<AnimationPlayer> player4 = timeline->createAnimationPlayer(0);
player4->setStartTimeInternal(20);
- RefPtr<AnimationPlayer> player5 = timeline->createAnimationPlayer(0);
+ RefPtrWillBeRawPtr<AnimationPlayer> player5 = timeline->createAnimationPlayer(0);
player5->setStartTimeInternal(10);
- RefPtr<AnimationPlayer> player6 = timeline->createAnimationPlayer(0);
+ RefPtrWillBeRawPtr<AnimationPlayer> player6 = timeline->createAnimationPlayer(0);
player6->setStartTimeInternal(-10);
- Vector<RefPtr<AnimationPlayer> > players;
+ Vector<RefPtrWillBeMember<AnimationPlayer> > players;
players.append(player6);
players.append(player2);
players.append(player5);

Powered by Google App Engine
This is Rietveld 408576698