Web Animations: Fix CSS events to handle animations with very short durations

Source/core/animation/TimedItemTest.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 19 matching lines @@
 
 #include "config.h"
 #include "core/animation/TimedItem.h"
 
 #include <gtest/gtest.h>
 
 using namespace WebCore;
 
 namespace {
 
-class TestTimedItemEventDelegate : public TimedItemEventDelegate {
+class TestTimedItemEventDelegate : public TimedItem::EventDelegate {
 public:
-    void onEventCondition(bool wasInPlay, bool isInPlay, double previousIteration, double currentIteration) OVERRIDE
+    void onEventCondition(bool isFirstSample, TimedItem::Phase previousPhase, TimedItem::Phase phase, double previousIteration, double currentIteration) OVERRIDE
     {
         m_eventTriggered = true;
-        m_playStateChanged = wasInPlay != isInPlay;
-        m_iterationChanged = isInPlay && previousIteration != currentIteration;
-        if (isInPlay)
-            ASSERT(!isNull(currentIteration));
+        m_phaseChanged = previousPhase != phase;
+        m_iterationChanged = previousIteration != currentIteration;
 
     }
     void reset()
     {
         m_eventTriggered = false;
-        m_playStateChanged = false;
+        m_phaseChanged = false;
         m_iterationChanged = false;
     }
     bool eventTriggered() { return m_eventTriggered; }
-    bool playStateChanged() { return m_playStateChanged; }
+    bool phaseChanged() { return m_phaseChanged; }
     bool iterationChanged() { return m_iterationChanged; }
 
 private:
     bool m_eventTriggered;
-    bool m_playStateChanged;
+    bool m_phaseChanged;
     bool m_iterationChanged;
 };
 
 class TestTimedItem : public TimedItem {
 public:
     static PassRefPtr<TestTimedItem> create(const Timing& specified)
     {
         return adoptRef(new TestTimedItem(specified, new TestTimedItemEventDelegate()));
     }
 
     void updateInheritedTime(double time)
     {
         m_eventDelegate->reset();
         TimedItem::updateInheritedTime(time);
     }
 
-    void updateChildrenAndEffects(bool wasActiveOrInEffect) const FINAL OVERRIDE {
-    }
-
+    void updateChildrenAndEffects(bool wasActiveOrInEffect) const FINAL OVERRIDE { }
     void willDetach() { }
-
     TestTimedItemEventDelegate* eventDelegate() { return m_eventDelegate; }
 
 private:
     TestTimedItem(const Timing& specified, TestTimedItemEventDelegate* eventDelegate)
         : TimedItem(specified, adoptPtr(eventDelegate))
         , m_eventDelegate(eventDelegate)
     {
     }
 
     TestTimedItemEventDelegate* m_eventDelegate;
 };
 
 TEST(TimedItem, Sanity)
 {
     Timing timing;
     timing.hasIterationDuration = true;
     timing.iterationDuration = 2;
     RefPtr<TestTimedItem> timedItem = TestTimedItem::create(timing);
 
-    ASSERT_FALSE(timedItem->isCurrent());
-    ASSERT_FALSE(timedItem->isInEffect());
-    ASSERT_FALSE(timedItem->isInPlay());
-    ASSERT_TRUE(isNull(timedItem->currentIteration()));
     ASSERT_EQ(0, timedItem->startTime());
-    ASSERT_TRUE(isNull(timedItem->activeDuration()));
-    ASSERT_TRUE(isNull(timedItem->timeFraction()));
 
     timedItem->updateInheritedTime(0);
 
+    ASSERT_EQ(TimedItem::PhaseActive, timedItem->phase());
     ASSERT_TRUE(timedItem->isInPlay());
     ASSERT_TRUE(timedItem->isCurrent());
     ASSERT_TRUE(timedItem->isInEffect());
     ASSERT_EQ(0, timedItem->currentIteration());
     ASSERT_EQ(0, timedItem->startTime());
     ASSERT_EQ(2, timedItem->activeDuration());
     ASSERT_EQ(0, timedItem->timeFraction());
 
     timedItem->updateInheritedTime(1);
 
+    ASSERT_EQ(TimedItem::PhaseActive, timedItem->phase());
     ASSERT_TRUE(timedItem->isInPlay());
     ASSERT_TRUE(timedItem->isCurrent());
     ASSERT_TRUE(timedItem->isInEffect());
     ASSERT_EQ(0, timedItem->currentIteration());
     ASSERT_EQ(0, timedItem->startTime());
     ASSERT_EQ(2, timedItem->activeDuration());
     ASSERT_EQ(0.5, timedItem->timeFraction());
 
     timedItem->updateInheritedTime(2);
 
+    ASSERT_EQ(TimedItem::PhaseAfter, timedItem->phase());
     ASSERT_FALSE(timedItem->isInPlay());
     ASSERT_FALSE(timedItem->isCurrent());
     ASSERT_TRUE(timedItem->isInEffect());
     ASSERT_EQ(0, timedItem->currentIteration());
     ASSERT_EQ(0, timedItem->startTime());
     ASSERT_EQ(2, timedItem->activeDuration());
     ASSERT_EQ(1, timedItem->timeFraction());
 
     timedItem->updateInheritedTime(3);
 
+    ASSERT_EQ(TimedItem::PhaseAfter, timedItem->phase());
     ASSERT_FALSE(timedItem->isInPlay());
     ASSERT_FALSE(timedItem->isCurrent());
     ASSERT_TRUE(timedItem->isInEffect());
     ASSERT_EQ(0, timedItem->currentIteration());
     ASSERT_EQ(0, timedItem->startTime());
     ASSERT_EQ(2, timedItem->activeDuration());
     ASSERT_EQ(1, timedItem->timeFraction());
 }
 
 TEST(TimedItem, FillForwards)
@@ skipping 171 matching lines @@
     timedItem->updateInheritedTime(2.75);
     ASSERT_EQ(2, timedItem->currentIteration());
     ASSERT_EQ(0.25, timedItem->timeFraction());
 }
 
 TEST(TimedItem, ZeroDurationSanity)
 {
     Timing timing;
     RefPtr<TestTimedItem> timedItem = TestTimedItem::create(timing);
 
-    ASSERT_FALSE(timedItem->isInPlay());
-    ASSERT_FALSE(timedItem->isCurrent());
-    ASSERT_FALSE(timedItem->isInEffect());
-    ASSERT_TRUE(isNull(timedItem->currentIteration()));
     ASSERT_EQ(0, timedItem->startTime());
-    ASSERT_TRUE(isNull(timedItem->activeDuration()));
-    ASSERT_TRUE(isNull(timedItem->timeFraction()));
 
     timedItem->updateInheritedTime(0);
 
+    ASSERT_EQ(TimedItem::PhaseAfter, timedItem->phase());
     ASSERT_FALSE(timedItem->isInPlay());
     ASSERT_FALSE(timedItem->isCurrent());
     ASSERT_TRUE(timedItem->isInEffect());
     ASSERT_EQ(0, timedItem->currentIteration());
     ASSERT_EQ(0, timedItem->startTime());
     ASSERT_EQ(0, timedItem->activeDuration());
     ASSERT_EQ(1, timedItem->timeFraction());
 
     timedItem->updateInheritedTime(1);
 
+    ASSERT_EQ(TimedItem::PhaseAfter, timedItem->phase());
     ASSERT_FALSE(timedItem->isInPlay());
     ASSERT_FALSE(timedItem->isCurrent());
     ASSERT_TRUE(timedItem->isInEffect());
     ASSERT_EQ(0, timedItem->currentIteration());
     ASSERT_EQ(0, timedItem->startTime());
     ASSERT_EQ(0, timedItem->activeDuration());
     ASSERT_EQ(1, timedItem->timeFraction());
 }
 
 TEST(TimedItem, ZeroDurationFillForwards)
@@ skipping 174 matching lines @@
     ASSERT_EQ(1, timedItem->currentIteration());
     ASSERT_EQ(1, timedItem->timeFraction());
 }
 
 TEST(TimedItem, Events)
 {
     Timing timing;
     timing.hasIterationDuration = true;
     timing.iterationDuration = 1;
     timing.iterationCount = 2;
+    timing.startDelay = 1;
     RefPtr<TestTimedItem> timedItem = TestTimedItem::create(timing);
 
-    timedItem->updateInheritedTime(0.3);
+    // First sample
+    timedItem->updateInheritedTime(0.0);
     ASSERT_TRUE(timedItem->eventDelegate()->eventTriggered());
-    EXPECT_TRUE(timedItem->eventDelegate()->playStateChanged());
+
+    // Before start
+    timedItem->updateInheritedTime(0.5);
+    ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
+
+    // First iteration
+    timedItem->updateInheritedTime(1.5);
+    ASSERT_TRUE(timedItem->eventDelegate()->eventTriggered());
+    EXPECT_TRUE(timedItem->eventDelegate()->phaseChanged());
     EXPECT_TRUE(timedItem->eventDelegate()->iterationChanged());
 
-    timedItem->updateInheritedTime(0.6);
+    timedItem->updateInheritedTime(1.6);
     ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
 
-    timedItem->updateInheritedTime(1.5);
-    ASSERT_TRUE(timedItem->eventDelegate()->eventTriggered());
-    EXPECT_TRUE(timedItem->eventDelegate()->iterationChanged());
-    EXPECT_FALSE(timedItem->eventDelegate()->playStateChanged());
-
+    // Second iteration
     timedItem->updateInheritedTime(2.5);
     ASSERT_TRUE(timedItem->eventDelegate()->eventTriggered());
-    EXPECT_FALSE(timedItem->eventDelegate()->iterationChanged());
-    EXPECT_TRUE(timedItem->eventDelegate()->playStateChanged());
+    EXPECT_FALSE(timedItem->eventDelegate()->phaseChanged());
+    EXPECT_TRUE(timedItem->eventDelegate()->iterationChanged());
 
-    timedItem->updateInheritedTime(3);
+    timedItem->updateInheritedTime(2.6);
     ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
 
-    timedItem->updateInheritedTime(1.5);
+    // After end
+    timedItem->updateInheritedTime(3.5);
     ASSERT_TRUE(timedItem->eventDelegate()->eventTriggered());
+    EXPECT_TRUE(timedItem->eventDelegate()->phaseChanged());
     EXPECT_FALSE(timedItem->eventDelegate()->iterationChanged());
-    EXPECT_TRUE(timedItem->eventDelegate()->playStateChanged());
+
+    timedItem->updateInheritedTime(3.6);
+    ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
 }
 }