setValueCurveAtTime takes sequence<float> for curve

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

 /*
  * Copyright (C) 2010 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:
  *
  * 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
@@ skipping 451 matching lines @@
                                         ExceptionState& exception_state) {
   WarnIfOutsideRange("setTargetAtTime value", target);
   Handler().Timeline().SetTargetAtTime(target, time, time_constant,
                                        exception_state);
 
   // Don't update the histogram here.  It's not clear in normal usage if the
   // parameter value will actually reach |target|.
   return this;
 }
 
-AudioParam* AudioParam::setValueCurveAtTime(NotShared<DOMFloat32Array> curve,
+AudioParam* AudioParam::setValueCurveAtTime(const Vector<float>& curve,
                                             double time,
                                             double duration,
                                             ExceptionState& exception_state) {
-  float* curve_data = curve.View()->Data();
   float min = minValue();
   float max = maxValue();
 
-  // First, find any non-finite value in the curve and throw an exception if
-  // there are any.
-  for (unsigned k = 0; k < curve.View()->length(); ++k) {
-    float value = curve_data[k];
-
-    if (!std::isfinite(value)) {
-      exception_state.ThrowDOMException(
-          kV8TypeError, "The provided float value for the curve at element " +
-                            String::Number(k) +
-                            " is non-finite: " + String::Number(value));
-      return nullptr;
-    }
-  }
-
-  // Second, find the first value in the curve (if any) that is outside the
-  // nominal range.  It's probably not necessary to produce a warning on every
-  // value outside the nominal range.
-  for (unsigned k = 0; k < curve.View()->length(); ++k) {
-    float value = curve_data[k];
+  // Find the first value in the curve (if any) that is outside the
+  // nominal range.  It's probably not necessary to produce a warning
+  // on every value outside the nominal range.
+  for (unsigned k = 0; k < curve.size(); ++k) {
+    float value = curve[k];
 
     if (value < min || value > max) {
       WarnIfOutsideRange("setValueCurveAtTime value", value);
       break;
     }
   }
 
-  Handler().Timeline().SetValueCurveAtTime(curve.View(), time, duration,
+  Handler().Timeline().SetValueCurveAtTime(curve, time, duration,
                                            exception_state);
 
   // We could update the histogram with every value in the curve, due to
   // interpolation, we'll probably be missing many values.  So we don't update
   // the histogram.  setValueCurveAtTime is probably a fairly rare method
   // anyway.
   return this;
 }
 
 AudioParam* AudioParam::cancelScheduledValues(double start_time,
                                               ExceptionState& exception_state) {
   Handler().Timeline().CancelScheduledValues(start_time, exception_state);
   return this;
 }
 
 AudioParam* AudioParam::cancelAndHoldAtTime(double start_time,
                                             ExceptionState& exception_state) {
   Handler().Timeline().CancelAndHoldAtTime(start_time, exception_state);
   return this;
 }
 
 }  // namespace blink