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

Unified Diff: third_party/WebKit/Source/core/animation/TimingCalculationsTest.cpp

Issue 2030843002: Web Animations: Account for end delay in after phase active time (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add headers Created 4 years, 5 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 | « third_party/WebKit/Source/core/animation/TimingCalculations.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/animation/TimingCalculationsTest.cpp
diff --git a/third_party/WebKit/Source/core/animation/TimingCalculationsTest.cpp b/third_party/WebKit/Source/core/animation/TimingCalculationsTest.cpp
index 698e5671d1a5e96f63e77c3f8e0ba40ad0a0fc5f..86d4f1810f559dee16ccf59795b66aad03f137ee 100644
--- a/third_party/WebKit/Source/core/animation/TimingCalculationsTest.cpp
+++ b/third_party/WebKit/Source/core/animation/TimingCalculationsTest.cpp
@@ -101,20 +101,20 @@ TEST(AnimationTimingCalculationsTest, IterationTime)
{
Timing timing;
- // calculateIterationTime(iterationDuration, repeatedDuration, scaledActiveTime, startOffset, timing)
+ // calculateIterationTime(iterationDuration, repeatedDuration, scaledActiveTime, startOffset, phase, timing)
// if the scaled active time is null
- EXPECT_TRUE(isNull(calculateIterationTime(1, 1, nullValue(), 1, timing)));
+ EXPECT_TRUE(isNull(calculateIterationTime(1, 1, nullValue(), 1, AnimationEffect::PhaseActive, timing)));
// if (complex-conditions)...
- EXPECT_EQ(12, calculateIterationTime(12, 12, 12, 0, timing));
+ EXPECT_EQ(12, calculateIterationTime(12, 12, 12, 0, AnimationEffect::PhaseActive, timing));
// otherwise
timing.iterationCount = 10;
- EXPECT_EQ(5, calculateIterationTime(10, 100, 25, 4, timing));
- EXPECT_EQ(7, calculateIterationTime(11, 110, 29, 1, timing));
+ EXPECT_EQ(5, calculateIterationTime(10, 100, 25, 4, AnimationEffect::PhaseActive, timing));
+ EXPECT_EQ(7, calculateIterationTime(11, 110, 29, 1, AnimationEffect::PhaseActive, timing));
timing.iterationStart = 1.1;
- EXPECT_EQ(8, calculateIterationTime(12, 120, 20, 7, timing));
+ EXPECT_EQ(8, calculateIterationTime(12, 120, 20, 7, AnimationEffect::PhaseActive, timing));
}
TEST(AnimationTimingCalculationsTest, CurrentIteration)
« no previous file with comments | « third_party/WebKit/Source/core/animation/TimingCalculations.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698