Throw TypeError if easing string is invalid.

third_party/WebKit/Source/core/animation/ElementAnimation.h

Index: third_party/WebKit/Source/core/animation/ElementAnimation.h
diff --git a/third_party/WebKit/Source/core/animation/ElementAnimation.h b/third_party/WebKit/Source/core/animation/ElementAnimation.h
index 1ad9e7d6f0a856ba56e3abb8f9d758d83f45510c..4932e8ebe0327f15ecc19acf3d066c14bbdb709b 100644
--- a/third_party/WebKit/Source/core/animation/ElementAnimation.h
+++ b/third_party/WebKit/Source/core/animation/ElementAnimation.h
@@ -55,16 +55,20 @@ public:
         EffectModel* effect = EffectInput::convert(&element, effectInput, executionContext, exceptionState);
         if (exceptionState.hadException())
             return 0;
-        return animateInternal(element, effect, TimingInput::convert(duration));
+        Timing timing;
+        ASSERT(TimingInput::convert(duration, timing));

[alancutter (OOO until 2018), 2016/04/05 04:50:06]

ASSERTs don't get compiled into official builds, you must store the return value
separately if you want to assert on it.

[suzyh_UTC10 (ex-contributor), 2016/04/12 08:04:23]

Oooh, whoops. Good catch, fixed.

[suzyh_UTC10 (ex-contributor), 2016/04/12 08:59:21]

Hmmm, I'm getting a compile failure on
chromeos_amd64-generic_chromium_compile_only_ng:
error: unused variable 'success' [-Werror=unused-variable]
     bool success = TimingInput::convert(duration, timing);

I can rewrite this as
if (!TimingInput::convert(...))
   ASSERT_NOT_REACHED();
but I don't know if this is overkill. I'm also not sure if this is just a quirk
of this bot, because linux_chromium_rel_ng (for instance) compiled fine.

What do you think?

[Timothy Loh, 2016/04/12 13:21:12]

DCHECK should work (previously this was called ASSERT_UNUSED but with the
impending style merger this has been deprecated in favor of DCHECK)

+        return animateInternal(element, effect, timing);
     }
 
     static Animation* animate(ExecutionContext* executionContext, Element& element, const EffectModelOrDictionarySequenceOrDictionary& effectInput, const KeyframeEffectOptions& options, ExceptionState& exceptionState)
     {
         EffectModel* effect = EffectInput::convert(&element, effectInput, executionContext, exceptionState);
-        if (exceptionState.hadException())
+        Timing timing;
+        bool success = TimingInput::convert(options, timing, exceptionState);
+        if (!success || exceptionState.hadException())
             return 0;
 
-        Animation* animation = animateInternal(element, effect, TimingInput::convert(options));
+        Animation* animation = animateInternal(element, effect, timing);
         animation->setId(options.id());
         return animation;
     }