[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
https://bugs.webkit.org/show_bug.cgi?id=67872

Attachment 106934: Patch
https://bugs.webkit.org/attachment.cgi?id=106934&action=review

------- 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