Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(767)

Unified Diff: third_party/WebKit/Source/modules/webaudio/AudioParam.cpp

Issue 1928163002: Add a name attribute for each AudioParam (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use enum instead of string Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/webaudio/AudioParam.cpp
diff --git a/third_party/WebKit/Source/modules/webaudio/AudioParam.cpp b/third_party/WebKit/Source/modules/webaudio/AudioParam.cpp
index aa505aa81ac9e5770e7ea99a51041d463cbdd79e..129df7a70bc5fd05c3b3c623c160681c7a75c145 100644
--- a/third_party/WebKit/Source/modules/webaudio/AudioParam.cpp
+++ b/third_party/WebKit/Source/modules/webaudio/AudioParam.cpp
@@ -40,6 +40,46 @@ AudioDestinationHandler& AudioParamHandler::destinationHandler() const
return *m_destinationHandler;
}
+String AudioParamHandler::getParamName() const
+{
+ // The returned string should be the name of the node and the name of the AudioParam for
+ // that node.
+ switch (m_paramType) {
+ case AudioBufferSourcePlaybackRate:
+ return "AudioBufferSource.playbackrate";
+ case AudioBufferSourceDetune:
+ return "AudioBufferSource.detune";
+ case BiquadFilterFrequency:
+ return "BiquadFilter.frequency";
+ case BiquadFilterQ:
+ return "BiquadFilter.Q";
+ case BiquadFilterGain:
+ return "BiquadFilter.gain";
+ case BiquadFilterDetune:
+ return "BiquadFilter.detune";
+ case DelayDelayTime:
+ return "Delay.delayTime";
+ case DynamicsCompressorThreshold:
+ return "DynamicsCompressor.threshold";
+ case DynamicsCompressorKnee:
+ return "DynamicsCompressor.knee";
+ case DynamicsCompressorRatio:
+ return "DynamicsCompressor.ratio";
+ case DynamicsCompressorAttack:
+ return "DynamicsCompressor.attack";
+ case DynamicsCompressorRelease:
+ return "DynamicsCompressor.release";
+ case GainGain:
+ return "Gain.gain";
+ case OscillatorFrequency:
+ return "Oscillator.frequency";
+ case OscillatorDetune:
+ return "Oscillator.detune";
+ case StereoPannerPan:
+ return "StereoPanner.pan";
+ };
+}
+
float AudioParamHandler::value()
{
// Update value for timeline.
@@ -193,15 +233,15 @@ void AudioParamHandler::disconnect(AudioNodeOutput& output)
// ----------------------------------------------------------------
-AudioParam::AudioParam(AbstractAudioContext& context, double defaultValue)
- : m_handler(AudioParamHandler::create(context, defaultValue))
+AudioParam::AudioParam(AbstractAudioContext& context, AudioParamHandler::ParamType paramType, double defaultValue)
+ : m_handler(AudioParamHandler::create(context, paramType, defaultValue))
, m_context(context)
{
}
-AudioParam* AudioParam::create(AbstractAudioContext& context, double defaultValue)
+AudioParam* AudioParam::create(AbstractAudioContext& context, AudioParamHandler::ParamType paramType, double defaultValue)
{
- return new AudioParam(context, defaultValue);
+ return new AudioParam(context, paramType, defaultValue);
}
DEFINE_TRACE(AudioParam)

Powered by Google App Engine
This is Rietveld 408576698