[webkit-reviews] review granted: [Bug 67872] DelayNode delay buffer is not correctly wrapping around : [Attachment 106934] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Sep 10 14:45:14 PDT 2011

Kenneth Russell <kbr at google.com> has granted Chris Rogers
<crogers at google.com>'s request for review:
Bug 67872: DelayNode delay buffer is not correctly wrapping around

Attachment 106934: Patch

------- Additional Comments from Kenneth Russell <kbr at google.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=106934&action=review

The test case is awesome. r=me

> LayoutTests/webaudio/delaynode.html:37
> +    }

Even though this is just a layout test, WebKit style is no braces for
single-line statements.

> LayoutTests/webaudio/delaynode.html:82
> +    }

Braces on single-line statements.

More information about the webkit-reviews mailing list