RESOLVED FIXED 46533
Add HighPass2FilterNode files 
https://bugs.webkit.org/show_bug.cgi?id=46533
Summary Add HighPass2FilterNode files
Chris Rogers
Reported 2010-09-24 15:40:54 PDT
Add HighPass2FilterNode files
Attachments
Patch (7.86 KB, patch)
2010-09-24 15:41 PDT, Chris Rogers 		no flags
DetailsFormatted DiffDiff
Patch (7.26 KB, patch)
2010-10-04 14:52 PDT, Chris Rogers 		no flags
DetailsFormatted DiffDiff
Patch (7.25 KB, patch)
2010-10-04 15:29 PDT, Chris Rogers 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Chris Rogers
Comment 1 2010-09-24 15:41:51 PDT
Created attachment 68775 [details] Patch
Chris Rogers
Comment 2 2010-10-04 14:52:13 PDT
Created attachment 69691 [details] Patch
Chris Rogers
Comment 3 2010-10-04 14:53:32 PDT
Fixed license and added FIXME in .idl about moving to BiquadNode.
Kenneth Russell
Comment 4 2010-10-04 15:00:28 PDT
Comment on attachment 69691 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=69691&action=review > WebCore/webaudio/HighPass2FilterNode.cpp:38 > + m_processor = biquadProcessor.release(); Please see LowPass2FilterNode for request about the disposition of the m_biquadProcessor member.
Chris Rogers
Comment 5 2010-10-04 15:29:05 PDT
Created attachment 69699 [details] Patch
Chris Rogers
Comment 6 2010-10-04 15:30:20 PDT
FIXED: Addressed comments about m_biquadProcessor as in LowPass2FilterNode.
Kenneth Russell
Comment 7 2010-10-04 15:50:01 PDT
Comment on attachment 69699 [details] Patch Looks good to me.
WebKit Commit Bot
Comment 8 2010-10-05 00:33:28 PDT
Comment on attachment 69699 [details] Patch Clearing flags on attachment: 69699 Committed r69088: <http://trac.webkit.org/changeset/69088>
WebKit Commit Bot
Comment 9 2010-10-05 00:33:35 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.