SVG animate should respect attribute 'restart=never'.

Source/core/svg/animation/SVGSMILElement.cpp

 /*
  * Copyright (C) 2008 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 925 matching lines @@
     return false;
 }
 
 SMILTime SVGSMILElement::nextProgressTime() const
 {
     return m_nextProgressTime;
 }
 
 void SVGSMILElement::beginListChanged(SMILTime eventTime)
 {
+    if (!m_isWaitingForFirstInterval && m_activeState != Active && this->restart() == RestartNever) {

[fs, 2015/05/04 14:46:52]

This will naturally fall into the else-branch below. Also, what if m_activeState
== Active and predicate below (in the else-branch is true?)

[Shanmuga Pandi, 2015/05/04 15:21:31]

Yes. If animation is currently active,  m_nextProgressTime and m_interval should
keep the current animation information. We need to reset back only if animation
got over. 


Correct me if I wrong. 

[fs, 2015/05/04 15:37:11]

...which will depend on what findInstanceTime() returns - no?

[Shanmuga Pandi, 2015/05/05 06:53:26]

When I checked with sample test case, findInstanceTime returns same eventTime
which is passed. And 
if (newBegin.isFinite() && (m_interval.end <= eventTime || newBegin < m_
      interval.begin) never been true when Animation is active.

Let me know if you want me to check some specific scenario which I missed?

+        m_nextProgressTime = SMILTime::unresolved();
+        m_interval.begin = SMILTime::unresolved();
+        m_interval.end = SMILTime::unresolved();
+        return;
+    }
+
     if (m_isWaitingForFirstInterval)
         resolveFirstInterval();
     else {
         SMILTime newBegin = findInstanceTime(Begin, eventTime, true);
         if (newBegin.isFinite() && (m_interval.end <= eventTime || newBegin < m_interval.begin)) {
             // Begin time changed, re-resolve the interval.
             SMILTime oldBegin = m_interval.begin;
             m_interval.end = eventTime;
             m_interval = resolveInterval(NextInterval);
             ASSERT(!m_interval.begin.isUnresolved());
@@ skipping 397 matching lines @@
     visitor->trace(m_targetElement);
     visitor->trace(m_timeContainer);
     visitor->trace(m_conditions);
     visitor->trace(m_syncBaseDependents);
 #endif
     SVGElement::trace(visitor);
     SVGTests::trace(visitor);
 }
 
 } // namespace blink