Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1193)

Unified Diff: Source/modules/webaudio/AudioBufferSourceNode.idl

Issue 26913005: start/stop method for AudioBufferSourceNodes and OscillatorNodes can take no args. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/webaudio/AudioBufferSourceNode.idl
diff --git a/Source/modules/webaudio/AudioBufferSourceNode.idl b/Source/modules/webaudio/AudioBufferSourceNode.idl
index 24c95eb3b2fbd93ce11b8be8c1c9bd05efa5d766..14cf8752558b82ee599ea5828b796baceea151bc 100644
--- a/Source/modules/webaudio/AudioBufferSourceNode.idl
+++ b/Source/modules/webaudio/AudioBufferSourceNode.idl
@@ -43,10 +43,10 @@
attribute double loopStart;
attribute double loopEnd;
- [MeasureAs=WebAudioStart] void start(double when);
+ [MeasureAs=WebAudioStart] void start(optional double when);
Ken Russell (switch to Gerrit) 2013/10/11 20:19:06 It's a little weird to use "optional" just for thi
Raymond Toy (Google) 2013/10/11 20:51:29 Let me try this out.
[MeasureAs=WebAudioStart, ImplementedAs=startGrain] void start(double when, double grainOffset);
[MeasureAs=WebAudioStart, ImplementedAs=startGrain] void start(double when, double grainOffset, double grainDuration);
- void stop(double when);
+ void stop(optional double when);
[DeprecateAs=WebAudioLooping, ImplementedAs=loop] attribute boolean looping; // This is an alias for the .loop attribute for backwards compatibility.

Powered by Google App Engine
This is Rietveld 408576698