[SVG] Handle repeatEvent for svg animations

Source/core/svg/animation/SVGSMILElement.cpp

Index: Source/core/svg/animation/SVGSMILElement.cpp
diff --git a/Source/core/svg/animation/SVGSMILElement.cpp b/Source/core/svg/animation/SVGSMILElement.cpp
index 0a2596505d2ee7205a6ae00cd0068455fa2488dd..93a02249371868fa005768d12c8dbce34bf2f633 100644
--- a/Source/core/svg/animation/SVGSMILElement.cpp
+++ b/Source/core/svg/animation/SVGSMILElement.cpp
@@ -57,6 +57,12 @@ static SMILEventSender& smilBeginEventSender()
     return sender;
 }
 
+static SMILEventSender& smilRepeatEventSender()
+{
+    DEFINE_STATIC_LOCAL(SMILEventSender, sender, ("repeatEvent"));
+    return sender;
+}
+
 // This is used for duration type time values that can't be negative.
 static const double invalidCachedTime = -1.;
 
@@ -148,6 +154,7 @@ SVGSMILElement::~SVGSMILElement()
     clearResourceReferences();
     smilEndEventSender().cancelEvent(this);
     smilBeginEventSender().cancelEvent(this);
+    smilRepeatEventSender().cancelEvent(this);
     disconnectConditions();
     if (m_timeContainer && m_targetElement && hasValidAttributeName())
         m_timeContainer->unschedule(this, m_targetElement, m_attributeName);
@@ -1117,6 +1124,9 @@ bool SVGSMILElement::progress(SMILTime elapsed, SVGSMILElement* resultElement, b
             startedActiveInterval();
         }
 
+        if (repeat && repeat != m_lastRepeat)
+            smilRepeatEventSender().dispatchEventSoon(this);
+
         updateAnimation(percent, repeat, resultElement);
         m_lastPercent = percent;
         m_lastRepeat = repeat;
@@ -1133,6 +1143,15 @@ bool SVGSMILElement::progress(SMILTime elapsed, SVGSMILElement* resultElement, b
     if (seekToTime) {
         if (m_activeState == Inactive)
             smilBeginEventSender().dispatchEventSoon(this);
+
+        if (repeat) {
+            for (int repeatEventCount = 1; repeatEventCount < repeat; repeatEventCount++) {

[pdr., 2013/08/31 04:48:29]

Looks like this is failing to compile on windows.

Can you change "int" here to "unsigned" to match the type of repeat?

+                smilRepeatEventSender().dispatchEventSoon(this);
+            }
+            if (m_activeState == Inactive)
+                smilRepeatEventSender().dispatchEventSoon(this);
+        }
+
         if (m_activeState == Inactive || m_activeState == Frozen)
             smilEndEventSender().dispatchEventSoon(this);
     }
@@ -1215,7 +1234,7 @@ void SVGSMILElement::endedActiveInterval()
 
 void SVGSMILElement::dispatchPendingEvent(SMILEventSender* eventSender)
 {
-    ASSERT(eventSender == &smilEndEventSender() || eventSender == &smilBeginEventSender());
+    ASSERT(eventSender == &smilEndEventSender() || eventSender == &smilBeginEventSender() || eventSender == &smilRepeatEventSender());
     const AtomicString& eventType = eventSender->eventType();
     dispatchEvent(Event::create(eventType));
 }