Summary: | BiquadFilterNode must not reset filter state if type attribute is set to the same type | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Rogers <crogers> | ||||
Component: | New Bugs | Assignee: | Chris Rogers <crogers> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | kbr, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Chris Rogers
2011-10-05 17:26:41 PDT
Created attachment 109888 [details]
Patch
http://trac.webkit.org/changeset/96525 is causing glitches in some cases because a BiquadFilterNode .type attribute is set redundantly (to the same value multiple times) in some web audio pages. When this happens, the filter state is being reset (causing the glitch). The filter state should only be reset if the filter type is actually changing to a different type, in which case it is appropriate. Comment on attachment 109888 [details]
Patch
Looks OK.
Comment on attachment 109888 [details] Patch Clearing flags on attachment: 109888 Committed r96846: <http://trac.webkit.org/changeset/96846> All reviewed patches have been landed. Closing bug. |