Implement optional arguments in AudioBufferSourceNode start() method
Created attachment 171727 [details] Patch
Comment on attachment 171727 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=171727&action=review Looks good. One minor comment. r=me > LayoutTests/webaudio/audiobuffersource-start.html:62 > + if (test.offsetFrame == "none" && test.durationFrames == "none") { You could consider leaving these properties undefined and check for "=== undefined". Not a big deal either way, but a simple typo in the string "none" could cause bugs.
Comment on attachment 171727 [details] Patch Thanks Ken, leaving as "none" for now just to make it more clear in the test vectors exactly what's going on
Comment on attachment 171727 [details] Patch Clearing flags on attachment: 171727 Committed r133101: <http://trac.webkit.org/changeset/133101>
All reviewed patches have been landed. Closing bug.