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

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

Issue 176903004: Have Document::timeline() / Document::transitionTimeline() return a reference (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 10 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 | « no previous file | Source/core/animation/DocumentAnimation.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/animation/AnimationTest.cpp
diff --git a/Source/core/animation/AnimationTest.cpp b/Source/core/animation/AnimationTest.cpp
index c530bf2030df6af2a1a61155505229285068fda1..beaae605149f07786aca0f6cc05fe5247f832345 100644
--- a/Source/core/animation/AnimationTest.cpp
+++ b/Source/core/animation/AnimationTest.cpp
@@ -25,8 +25,8 @@ protected:
, element(document->createElement("foo", ASSERT_NO_EXCEPTION))
{
document->animationClock().resetTimeForTesting();
- document->timeline()->setZeroTime(0);
- EXPECT_EQ(0, document->timeline()->currentTime());
+ document->timeline().setZeroTime(0);
+ EXPECT_EQ(0, document->timeline().currentTime());
}
RefPtr<Document> document;
@@ -286,7 +286,7 @@ TEST_F(AnimationAnimationTest, TimeToEffectChange)
timing.endDelay = 100;
timing.fillMode = Timing::FillModeNone;
RefPtr<Animation> animation = Animation::create(nullptr, nullptr, timing);
- RefPtr<Player> player = document->timeline()->play(animation.get());
+ RefPtr<Player> player = document->timeline().play(animation.get());
double inf = std::numeric_limits<double>::infinity();
EXPECT_EQ(100, animation->timeToForwardsEffectChange());
@@ -319,7 +319,7 @@ TEST_F(AnimationAnimationTest, TimeToEffectChangeWithPlaybackRate)
timing.playbackRate = 2;
timing.fillMode = Timing::FillModeNone;
RefPtr<Animation> animation = Animation::create(nullptr, nullptr, timing);
- RefPtr<Player> player = document->timeline()->play(animation.get());
+ RefPtr<Player> player = document->timeline().play(animation.get());
double inf = std::numeric_limits<double>::infinity();
EXPECT_EQ(100, animation->timeToForwardsEffectChange());
@@ -352,7 +352,7 @@ TEST_F(AnimationAnimationTest, TimeToEffectChangeWithNegativePlaybackRate)
timing.playbackRate = -2;
timing.fillMode = Timing::FillModeNone;
RefPtr<Animation> animation = Animation::create(nullptr, nullptr, timing);
- RefPtr<Player> player = document->timeline()->play(animation.get());
+ RefPtr<Player> player = document->timeline().play(animation.get());
double inf = std::numeric_limits<double>::infinity();
EXPECT_EQ(100, animation->timeToForwardsEffectChange());
« no previous file with comments | « no previous file | Source/core/animation/DocumentAnimation.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698