Add min/max values for AudioParams

third_party/WebKit/Source/modules/webaudio/AudioBufferSourceNode.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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 565 matching lines @@
             // playing. Stop the node now and deref it. (But don't run the onEnded event because the
             // source never actually played.)
             finishWithoutOnEnded();
         }
     }
 }
 
 // ----------------------------------------------------------------
 AudioBufferSourceNode::AudioBufferSourceNode(AbstractAudioContext& context, float sampleRate)
     : AudioScheduledSourceNode(context)
-    , m_playbackRate(AudioParam::create(context, ParamTypeAudioBufferSourcePlaybackRate, 1.0))
+    // Arbitrarily choose a limit of +/- Nyquist frequency for playback

[hongchan, 2016/05/13 01:30:14]

Not arbitrarily - it is based on the spec. Right?

[Raymond Toy, 2016/05/13 17:06:15]

Forgot to update it after it was decided to be unlimited.

[Raymond Toy, 2016/05/13 18:40:32]

Done.

+    , m_playbackRate(AudioParam::create(context, ParamTypeAudioBufferSourcePlaybackRate, 1.0, -sampleRate / 2, sampleRate / 2))
     , m_detune(AudioParam::create(context, ParamTypeAudioBufferSourceDetune, 0.0))
 {
     setHandler(AudioBufferSourceHandler::create(*this, sampleRate, m_playbackRate->handler(), m_detune->handler()));
 }
 
 AudioBufferSourceNode* AudioBufferSourceNode::create(AbstractAudioContext& context, float sampleRate)
 {
     return new AudioBufferSourceNode(context, sampleRate);
 }
 
@@ skipping 73 matching lines @@
 {
     audioBufferSourceHandler().start(when, grainOffset, exceptionState);
 }
 
 void AudioBufferSourceNode::start(double when, double grainOffset, double grainDuration, ExceptionState& exceptionState)
 {
     audioBufferSourceHandler().start(when, grainOffset, grainDuration, exceptionState);
 }
 
 } // namespace blink