Web Animations: Throw exceptions for play/pause/reverse/finish with infinite end time

third_party/WebKit/Source/core/animation/Animation.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 482 matching lines @@
         return Paused;
     if (m_playState == Idle)
         return Idle;
     if (m_currentTimePending || (isNull(m_startTime) && m_playbackRate != 0))
         return Pending;
     if (limited())
         return Finished;
     return Running;
 }
 
-void Animation::pause()
+void Animation::pause(ExceptionState& exceptionState)
 {
     if (m_paused)
         return;
 
     PlayStateUpdateScope updateScope(*this, TimingUpdateOnDemand);
 
     double newCurrentTime = currentTimeInternal();
     if (calculatePlayState() == Idle) {
+        if (m_playbackRate < 0 && effectEnd() == std::numeric_limits<double>::infinity()) {
+            exceptionState.throwDOMException(InvalidStateError, "Cannot pause, Animation has infinite target effect end.");
+            return;
+        }
         newCurrentTime = m_playbackRate < 0 ? effectEnd() : 0;
     }
 
     m_playState = Unset;
     m_paused = true;
     m_currentTimePending = true;
     setCurrentTimeInternal(newCurrentTime, TimingUpdateOnDemand);
 }
 
 void Animation::unpause()
 {
     if (!m_paused)
         return;
 
     PlayStateUpdateScope updateScope(*this, TimingUpdateOnDemand);
 
     m_currentTimePending = true;
     unpauseInternal();
 }
 
 void Animation::unpauseInternal()
 {
     if (!m_paused)
         return;
     m_paused = false;
     setCurrentTimeInternal(currentTimeInternal(), TimingUpdateOnDemand);
 }
 
-void Animation::play()
+void Animation::play(ExceptionState& exceptionState)
 {
     PlayStateUpdateScope updateScope(*this, TimingUpdateOnDemand);
 
+    double currentTime = this->currentTimeInternal();
+    if (m_playbackRate < 0 && currentTime <= 0 && effectEnd() == std::numeric_limits<double>::infinity()) {
+        exceptionState.throwDOMException(InvalidStateError, "Cannot play reversed Animation with infinite target effect end.");
+        return;
+    }
+
     if (!playing()) {
         m_startTime = nullValue();
     }
 
     if (playStateInternal() == Idle) {
         m_held = true;
         m_holdTime = 0;
     }
 
-    double currentTime = this->currentTimeInternal();
-
     m_playState = Unset;
     m_finished = false;
     unpauseInternal();
 
     if (m_playbackRate > 0 && (currentTime < 0 || currentTime >= effectEnd())) {
         m_startTime = nullValue();
         setCurrentTimeInternal(0, TimingUpdateOnDemand);
     } else if (m_playbackRate < 0 && (currentTime <= 0 || currentTime > effectEnd())) {
         m_startTime = nullValue();
         setCurrentTimeInternal(effectEnd(), TimingUpdateOnDemand);
     }
 }
 
-void Animation::reverse()
+void Animation::reverse(ExceptionState& exceptionState)
 {
     if (!m_playbackRate) {
         return;
     }
 
     setPlaybackRateInternal(-m_playbackRate);
-    play();
+    play(exceptionState);
 }
 
 void Animation::finish(ExceptionState& exceptionState)
 {
     PlayStateUpdateScope updateScope(*this, TimingUpdateOnDemand);
 
     if (!m_playbackRate) {
         exceptionState.throwDOMException(InvalidStateError, "Cannot finish Animation with a playbackRate of 0.");
         return;
     }
@@ skipping 483 matching lines @@
     visitor->trace(m_timeline);
     visitor->trace(m_pendingFinishedEvent);
     visitor->trace(m_pendingCancelledEvent);
     visitor->trace(m_finishedPromise);
     visitor->trace(m_readyPromise);
     RefCountedGarbageCollectedEventTargetWithInlineData<Animation>::trace(visitor);
     ActiveDOMObject::trace(visitor);
 }
 
 } // namespace blink