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

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

Issue 1934683002: Add histograms for Biquad lowpass and highpass Q values (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase 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.h
diff --git a/third_party/WebKit/Source/modules/webaudio/AudioParam.h b/third_party/WebKit/Source/modules/webaudio/AudioParam.h
index 42bffe793b960d2e3a94a66092b39c76a47f341a..e8f4999a9bf9cc55933aaf8521a8a9e43b1aaa45 100644
--- a/third_party/WebKit/Source/modules/webaudio/AudioParam.h
+++ b/third_party/WebKit/Source/modules/webaudio/AudioParam.h
@@ -51,6 +51,8 @@ enum AudioParamType {
ParamTypeAudioBufferSourceDetune,
ParamTypeBiquadFilterFrequency,
ParamTypeBiquadFilterQ,
+ ParamTypeBiquadFilterQLowpass,
+ ParamTypeBiquadFilterQHighpass,
ParamTypeBiquadFilterGain,
ParamTypeBiquadFilterDetune,
ParamTypeDelayDelayTime,
@@ -75,6 +77,7 @@ enum AudioParamType {
class AudioParamHandler final : public ThreadSafeRefCounted<AudioParamHandler>, public AudioSummingJunction {
public:
AudioParamType getParamType() const { return m_paramType; }
+ void setParamType(AudioParamType);
// Return a nice name for the AudioParam.
String getParamName() const;
@@ -127,6 +130,10 @@ public:
void disconnect(AudioNodeOutput&);
float intrinsicValue() const { return noBarrierLoad(&m_intrinsicValue); }
+
+ // Update any histograms with the given value.
+ void updateHistograms(float newValue);
+
private:
AudioParamHandler(AbstractAudioContext& context, AudioParamType paramType, double defaultValue)
: AudioSummingJunction(context.deferredTaskHandler())
@@ -172,6 +179,7 @@ public:
AbstractAudioContext* context() const { return m_context; }
AudioParamType getParamType() const { return handler().getParamType(); }
+ void setParamType(AudioParamType);
String getParamName() const;
float value() const;

Powered by Google App Engine
This is Rietveld 408576698