[Webkit-unassigned] [Bug 82200] Reset paramters when BiquadFilterNode.type get new value.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=82200





--- Comment #2 from Chris Rogers <crogers at google.com>  2012-03-26 22:09:12 PST ---
(From update of attachment 133835)
View in context: https://bugs.webkit.org/attachment.cgi?id=133835&action=review

Hi Chun, thanks for the patch.  I don't think we should change the AudioParams in this way, but a good change would be to remove the deprecated filter nodes.

> Source/WebCore/Modules/webaudio/BiquadProcessor.cpp:-48
> -    m_parameter3 = AudioParam::create("gain", 0.0, -40, 40);

The attributes for BiquadFilterNode are defined in the spec as "frequency", "Q", "gain", so I think we shouldn't remove this code here, even though for some types "Q" is unused.

> Source/WebCore/Modules/webaudio/BiquadProcessor.cpp:-76
> -    }

I think it may be a good time to think about removing these old/deprecated AudioNodes (LowPass2FilterNode and HighPass2FilterNode).  There have been warning messages for many months.  But that should probably be in a different patch called "Remove deprecated LowPass2FilterNode and HighPass2FilterNode"

> Source/WebCore/Modules/webaudio/BiquadProcessor.cpp:157
> +}

I don't think we should re-create these AudioParams -- instead just create in the normal constructor and leave named the same always

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.