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 21 matching lines @@
 #include "core/animation/TimedItem.h"
 
 #include <gtest/gtest.h>
 
 using namespace WebCore;
 
 namespace {
 
 class TestTimedItemEventDelegate : public TimedItemEventDelegate {
 public:
-    void onEventCondition(bool wasInPlay, bool isInPlay, double previousIteration, double currentIteration) OVERRIDE
+    void onEventCondition(bool wasInPlay, bool isInPlay, bool wasCurrent, bool isCurrent, double previousIteration, double currentIteration) OVERRIDE
     {
         m_eventTriggered = true;
-        m_playStateChanged = wasInPlay != isInPlay;
+        m_inPlayStateChanged = wasInPlay != isInPlay;
+        m_currentStateChanged = wasCurrent != isCurrent;
         m_iterationChanged = isInPlay && previousIteration != currentIteration;
         if (isInPlay)
             ASSERT(!isNull(currentIteration));
 
     }
     void reset()
     {
         m_eventTriggered = false;
-        m_playStateChanged = false;
+        m_inPlayStateChanged = false;
+        m_currentStateChanged = false;
         m_iterationChanged = false;
     }
     bool eventTriggered() { return m_eventTriggered; }
-    bool playStateChanged() { return m_playStateChanged; }
+    bool inPlayStateChanged() { return m_inPlayStateChanged; }
+    bool currentStateChanged() { return m_currentStateChanged; }
     bool iterationChanged() { return m_iterationChanged; }
 
 private:
     bool m_eventTriggered;
-    bool m_playStateChanged;
+    bool m_inPlayStateChanged;
+    bool m_currentStateChanged;
     bool m_iterationChanged;
 };
 
 class TestTimedItem : public TimedItem {
 public:
     static PassRefPtr<TestTimedItem> create(const Timing& specified)
     {
         return adoptRef(new TestTimedItem(specified, new TestTimedItemEventDelegate()));
     }
 
@@ skipping 20 matching lines @@
     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_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());
@@ skipping 204 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_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());
@@ skipping 187 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);
+    // Before start
+    timedItem->updateInheritedTime(0.0);
+    ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
+
+    timedItem->updateInheritedTime(0.5);
+    ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
+
+    // First iteration
+    timedItem->updateInheritedTime(1.5);
     ASSERT_TRUE(timedItem->eventDelegate()->eventTriggered());
-    EXPECT_TRUE(timedItem->eventDelegate()->playStateChanged());
+    EXPECT_TRUE(timedItem->eventDelegate()->inPlayStateChanged());
+    EXPECT_FALSE(timedItem->eventDelegate()->currentStateChanged());
     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()->inPlayStateChanged());
+    EXPECT_FALSE(timedItem->eventDelegate()->currentStateChanged());
+    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()->inPlayStateChanged());
+    EXPECT_TRUE(timedItem->eventDelegate()->currentStateChanged());
     EXPECT_FALSE(timedItem->eventDelegate()->iterationChanged());
-    EXPECT_TRUE(timedItem->eventDelegate()->playStateChanged());
+
+    timedItem->updateInheritedTime(3.6);
+    ASSERT_FALSE(timedItem->eventDelegate()->eventTriggered());
 }
 }