Oilpan: Completely move core/animations/ to oilpan's heap

Source/core/animation/AnimationPlayerTest.cpp

 /*
  * Copyright (c) 2013, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 50 matching lines @@
         player->setStartTimeInternal(0);
         player->setSource(makeAnimation().get());
     }
 
     void startTimeline()
     {
         timeline->setZeroTime(0);
         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;
         timing.playbackRate = playbackRate;
         return Animation::create(0, nullptr, timing);
     }
 
     bool updateTimeline(double time)
     {
         document->animationClock().updateTime(time);
         // The timeline does not know about our player, so we have to explicitly call update().
         return player->update(AnimationPlayer::UpdateOnDemand);
     }
 
     RefPtr<Document> document;
-    RefPtr<DocumentTimeline> timeline;
-    RefPtr<AnimationPlayer> player;
+    RefPtrWillBePersistent<DocumentTimeline> timeline;
+    RefPtrWillBePersistent<AnimationPlayer> player;
     TrackExceptionState exceptionState;
 };
 
 TEST_F(AnimationAnimationPlayerTest, InitialState)
 {
     setUpWithoutStartingTimeline();
     player = timeline->createAnimationPlayer(0);
     EXPECT_TRUE(isNull(timeline->currentTimeInternal()));
     EXPECT_EQ(0, player->currentTimeInternal());
     EXPECT_FALSE(player->paused());
@@ skipping 456 matching lines @@
     EXPECT_EQ(0, player->currentTimeInternal());
     updateTimeline(1);
     EXPECT_EQ(30, player->currentTimeInternal());
 }
 
 
 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());
     player->setCurrentTimeInternal(15);
     player->setSource(source2.get());
     EXPECT_EQ(15, player->currentTimeInternal());
     EXPECT_EQ(0, source1->player());
     EXPECT_EQ(player.get(), source2->player());
     EXPECT_EQ(source2, player->source());
 }
@@ skipping 36 matching lines @@
     player->setSource(timedItem);
     EXPECT_EQ(0, player2->source());
 }
 
 TEST_F(AnimationAnimationPlayerTest, AnimationPlayersReturnTimeToNextEffect)
 {
     Timing timing;
     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);
 
     updateTimeline(0);
     EXPECT_EQ(1, player->timeToEffectChange());
 
     updateTimeline(0.5);
     EXPECT_EQ(0.5, player->timeToEffectChange());
 
     updateTimeline(1);
@@ skipping 56 matching lines @@
     player->cancel();
     player->update(AnimationPlayer::UpdateOnDemand);
     EXPECT_EQ(3, player->timeToEffectChange());
 }
 
 TEST_F(AnimationAnimationPlayerTest, AttachedAnimationPlayers)
 {
     RefPtr<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());
     timeline->serviceAnimations(AnimationPlayer::UpdateForAnimationFrame);
     EXPECT_EQ(1, element->activeAnimations()->players().find(player.get())->value);
 
     player.release();
     EXPECT_EQ(0, element->activeAnimations()->players().find(player.get())->value);
     EXPECT_TRUE(element->activeAnimations()->players().isEmpty());
 }
 
 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;

[Mads Ager (chromium), 2014/04/29 10:29:41]

WillBeHeapVector, otherwise it will not be traced.

     players.append(player6);
     players.append(player2);
     players.append(player5);
     players.append(player1);
     players.append(player3);
     players.append(player4);
     for (size_t i = 0; i < players.size(); i++) {
         for (size_t j = 0; j < players.size(); j++)
             EXPECT_EQ(i < j, AnimationPlayer::hasLowerPriority(players[i].get(), players[j].get()));
     }
 }
 
 }