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

Unified Diff: LayoutTests/webaudio/dom-exceptions.html

Issue 825603005: AudioBufferSourceNode.buffer can only be set once. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Use deprecation message instead of DOM error Created 5 years, 11 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: LayoutTests/webaudio/dom-exceptions.html
diff --git a/LayoutTests/webaudio/dom-exceptions.html b/LayoutTests/webaudio/dom-exceptions.html
index 517169284e82f9be20de3511e0edc2a45b2ec699..dc591a011bae82c9d5e8999967f8b23b163f8e7c 100644
--- a/LayoutTests/webaudio/dom-exceptions.html
+++ b/LayoutTests/webaudio/dom-exceptions.html
@@ -168,6 +168,9 @@ function runTest() {
buffer = context.createBuffer(1,1, context.sampleRate);
shouldNotThrow("source = context.createBufferSource()");
shouldNotThrow("source.buffer = buffer");
+ // Setting the buffer twice will eventually throw an error, but for now, it's just a deprecation
+ // warning.
+ shouldNotThrow("source.buffer = context.createBuffer(1, 10, context.sampleRate)");
shouldThrow("source.start(-1)");
shouldThrow("source.start(Infinity)");
shouldThrow("source.start(-Infinity)");

Powered by Google App Engine
This is Rietveld 408576698