Web Animations: Throw TypeErrors for invalid timing parameters

third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp

Index: third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp
diff --git a/third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp b/third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp
index 2591f79bbe7a802acddb68050c02bb21a86114ae..ec1f71328bd03718e02c6d700bcb23b21658ca0b 100644
--- a/third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp
+++ b/third_party/WebKit/Source/core/inspector/InspectorAnimationAgent.cpp
@@ -333,6 +333,7 @@ void InspectorAnimationAgent::setTiming(ErrorString* errorString, const String&
         return;
 
     animation = animationClone(animation);
+    NonThrowableExceptionState exceptionState;

[suzyh_UTC10 (ex-contributor), 2016/04/20 21:54:12]

I don't know the implications of this, so best to run it by samli if you haven't
already.

 
     String type = m_idToAnimationType.get(animationId);
     if (type == AnimationType::CSSTransition) {
@@ -352,12 +353,12 @@ void InspectorAnimationAgent::setTiming(ErrorString* errorString, const String&
         AnimationEffectTiming* timing = animation->effect()->timing();
         UnrestrictedDoubleOrString unrestrictedDuration;
         unrestrictedDuration.setUnrestrictedDouble(duration + delay);
-        timing->setDuration(unrestrictedDuration);
+        timing->setDuration(unrestrictedDuration, exceptionState);
     } else {
         AnimationEffectTiming* timing = animation->effect()->timing();
         UnrestrictedDoubleOrString unrestrictedDuration;
         unrestrictedDuration.setUnrestrictedDouble(duration);
-        timing->setDuration(unrestrictedDuration);
+        timing->setDuration(unrestrictedDuration, exceptionState);
         timing->setDelay(delay);
     }
 }