Drop [LegacyInterfaceTypeChecking] for the Web Audio API

third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp

Index: third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp b/third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp
index e16331e5d228db99a15e54f714fd67c23a6921ff..f4967991e7bafc8d3e735b62a90d4a5745c9d6cd 100644
--- a/third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/AudioParamTimeline.cpp
@@ -201,6 +201,7 @@ void AudioParamTimeline::setTargetAtTime(float target, double time, double timeC
 void AudioParamTimeline::setValueCurveAtTime(DOMFloat32Array* curve, double time, double duration, ExceptionState& exceptionState)
 {
     ASSERT(isMainThread());
+    ASSERT(curve);

[Raymond Toy, 2015/12/04 17:30:42]

In a release build, what should we do if curve is null?  This shouldn't happen,
but if it's worth putting in a debug assert we should think about the release
build.

Chris Rogers was fairly careful but silently returning on release builds for bad
situations like this.

[philipj_slow, 2015/12/04 18:31:57]

Would you like me to revert all changes to cpp files, or something else? In all
other cases where bindings guarantees that the input is not null, I've checked
that there are no internal calls and ASSERTed it, to make responsibilities
clear, but I will do whatever you're most comfortable with.

[Raymond Toy, 2015/12/04 21:06:26]

Thanks for the info.  I think this is fine, plus you've added additional tests
for this.

 
     if (!isNonNegativeAudioParamTime(time, exceptionState)
         || !isPositiveAudioParamTime(duration, exceptionState, "Duration"))