Web Animations CSS: Unfreeze AnimationClock if sampling timelines does not trigger style recalc

Source/core/animation/DocumentTimelineTest.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 96 matching lines @@
 
     void wake()
     {
         timeline->wake();
     }
 
     double minimumDelay()
     {
         return DocumentTimeline::s_minimumDelay;
     }
-
 };
 
 TEST_F(CoreAnimationDocumentTimelineTest, EmptyKeyframeAnimation)
 {
     RefPtr<KeyframeAnimationEffect> effect = KeyframeAnimationEffect::create(KeyframeAnimationEffect::KeyframeVector());
     RefPtr<Animation> anim = Animation::create(element.get(), effect, timing);
 
     timeline->play(anim.get());
 
     platformTiming->expectNoMoreActions();
     timeline->serviceAnimations(0);
     EXPECT_FLOAT_EQ(0, timeline->currentTime());
     EXPECT_TRUE(anim->compositableValues()->isEmpty());
 
     platformTiming->expectNoMoreActions();
     timeline->serviceAnimations(100);
     EXPECT_FLOAT_EQ(100, timeline->currentTime());
 }
 
-TEST_F(CoreAnimationDocumentTimelineTest, ZeroTimeAsPerfTime)
+TEST_F(CoreAnimationDocumentTimelineTest, EmptyTimelineDoesNotFreezeTime)
+{
+    timeline->serviceAnimations(100);
+    EXPECT_NE(100, timeline->currentTime());
+}
+
+TEST_F(CoreAnimationDocumentTimelineTest, EmptyPlayerDoesNotFreezeTime)
+{
+    timeline->play(0);
+    timeline->serviceAnimations(100);
+    EXPECT_NE(100, timeline->currentTime());
+}
+
+TEST_F(CoreAnimationDocumentTimelineTest, EmptyTargetDoesNotFreezeTime)
+{
+    timing.iterationDuration = 200;
+    timeline->play(Animation::create(0, KeyframeAnimationEffect::create(KeyframeAnimationEffect::KeyframeVector()), timing).get());
+    timeline->serviceAnimations(100);
+    EXPECT_NE(100, timeline->currentTime());
+}
+
+TEST_F(CoreAnimationDocumentTimelineTest, EmptyEffectDoesNotFreezeTime)
+{
+    timeline->play(Animation::create(element.get(), 0, timing).get());
+    timeline->serviceAnimations(100);
+    EXPECT_NE(100, timeline->currentTime());
+}
+
+TEST_F(CoreAnimationDocumentTimelineTest, FreezeTime)
+{
+    timeline->play(Animation::create(element.get(), KeyframeAnimationEffect::create(KeyframeAnimationEffect::KeyframeVector()), timing).get());
+    timeline->serviceAnimations(100);
+    EXPECT_EQ(100, timeline->currentTime());
+}
+
+TEST_F(CoreAnimationDocumentTimelineTest, ZeroTime)
 {
     timeline = DocumentTimeline::create(document.get());
 
-    timeline->serviceAnimations(100);
+    document->animationClock().updateTime(100);
     EXPECT_TRUE(isNull(timeline->currentTime()));
 
-    timeline->serviceAnimations(200);
+    document->animationClock().updateTime(200);
     EXPECT_TRUE(isNull(timeline->currentTime()));
 
     timeline->setZeroTime(300);
     document->animationClock().updateTime(300);
-    timeline->serviceAnimations(300);
     EXPECT_EQ(0, timeline->currentTime());
 
-    timeline->serviceAnimations(400);
+    document->animationClock().updateTime(400);
     EXPECT_EQ(100, timeline->currentTime());
 }
 
 TEST_F(CoreAnimationDocumentTimelineTest, PauseForTesting)
 {
     float seekTime = 1;
     RefPtr<Animation> anim1 = Animation::create(element.get(), KeyframeAnimationEffect::create(KeyframeAnimationEffect::KeyframeVector()), timing);
     RefPtr<Animation> anim2  = Animation::create(element.get(), KeyframeAnimationEffect::create(KeyframeAnimationEffect::KeyframeVector()), timing);
     RefPtr<Player> player1 = timeline->play(anim1.get());
     RefPtr<Player> player2 = timeline->play(anim2.get());
@@ skipping 45 matching lines @@
     platformTiming->expectNextFrameAction();
     timeline->serviceAnimations(1.5);
     EXPECT_EQ(4U, timeline->numberOfActiveAnimationsForTesting());
     platformTiming->expectNoMoreActions();
     timeline->serviceAnimations(3);
     EXPECT_EQ(1U, timeline->numberOfActiveAnimationsForTesting());
 }
 
 TEST_F(CoreAnimationDocumentTimelineTest, DelayBeforeAnimationStart)
 {
-
     timing.hasIterationDuration = true;
     timing.iterationDuration = 2;
     timing.startDelay = 5;
 
     RefPtr<Animation> anim = Animation::create(element.get(), 0, timing);
 
     RefPtr<Player> player = timeline->play(anim.get());
 
     // TODO: Put the player startTime in the future when we add the capability to change player startTime
     platformTiming->expectDelayedAction(timing.startDelay - minimumDelay());
     timeline->serviceAnimations(0);
 
     platformTiming->expectDelayedAction(timing.startDelay - minimumDelay() - 1.5);
     timeline->serviceAnimations(1.5);
 
     EXPECT_CALL(*platformTiming, serviceOnNextFrame());
     wake();
 
     platformTiming->expectNextFrameAction();
     timeline->serviceAnimations(4.98);
 }
 
 }