Have Player take the DocumentTimeline object by reference

Source/core/animation/PlayerTest.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 32 matching lines @@
 
 namespace {
 
 class CoreAnimationPlayerTest : public ::testing::Test {
 protected:
     virtual void SetUp()
     {
         document = Document::create();
         document->animationClock().resetTimeForTesting();
         timeline = DocumentTimeline::create(document.get());
-        player = Player::create(timeline.get(), 0);
+        player = Player::create(*timeline, 0);
         timeline->setZeroTime(0);
     }
 
     bool updateTimeline(double time, double* timeToEffectChange = 0)
     {
         timeline->serviceAnimations(time);
         // The timeline does not know about our player, so we have to explicitly call update().
         return player->update(timeToEffectChange);
     }
 
     RefPtr<Document> document;
     RefPtr<DocumentTimeline> timeline;
     RefPtr<Player> player;
 };
 
 TEST_F(CoreAnimationPlayerTest, InitialState)
 {
     EXPECT_EQ(0, timeline->currentTime());
     EXPECT_EQ(0, player->currentTime());
     EXPECT_FALSE(player->paused());
     EXPECT_EQ(1, player->playbackRate());
     EXPECT_EQ(0, player->startTime());
     EXPECT_EQ(0, player->timeDrift());
 }
 
 TEST_F(CoreAnimationPlayerTest, CreatePlayerAfterTimelineStarted)
 {
     updateTimeline(1234);
     EXPECT_EQ(1234, timeline->currentTime());
-    RefPtr<Player> player = Player::create(timeline.get(), 0);
+    RefPtr<Player> player = Player::create(*timeline, 0);
     EXPECT_EQ(1234, player->startTime());
     EXPECT_EQ(0, player->currentTime());
 }
 
 TEST_F(CoreAnimationPlayerTest, PauseUnpause)
 {
     updateTimeline(200);
     player->setPaused(true);
     EXPECT_TRUE(player->paused());
     EXPECT_EQ(200, player->currentTime());
@@ skipping 178 matching lines @@
     EXPECT_EQ(std::numeric_limits<double>::infinity(), timeToNextEffect);
 }
 
 TEST_F(CoreAnimationPlayerTest, PlayersReturnTimeToNextEffect)
 {
     Timing timing;
     timing.startDelay = 1;
     timing.iterationDuration = 1;
     timing.hasIterationDuration = true;
     RefPtr<Animation> animation = Animation::create(0, 0, timing);
-    player = Player::create(timeline.get(), animation.get());
+    player = Player::create(*timeline, animation.get());
 
     double timeToNextEffect;
     updateTimeline(0, &timeToNextEffect);
     EXPECT_EQ(1, timeToNextEffect);
 
     updateTimeline(0.5, &timeToNextEffect);
     EXPECT_EQ(0.5, timeToNextEffect);
 
     updateTimeline(1, &timeToNextEffect);
     EXPECT_EQ(0, timeToNextEffect);
 
     updateTimeline(1.5, &timeToNextEffect);
     EXPECT_EQ(0, timeToNextEffect);
 
     updateTimeline(2, &timeToNextEffect);
     EXPECT_EQ(std::numeric_limits<double>::infinity(), timeToNextEffect);
 
     updateTimeline(3, &timeToNextEffect);
     EXPECT_EQ(std::numeric_limits<double>::infinity(), timeToNextEffect);
 }
 
 }