ConvolverNode.buffer sample rate must match AudioContext rate

LayoutTests/webaudio/dom-exceptions.html

Index: LayoutTests/webaudio/dom-exceptions.html
diff --git a/LayoutTests/webaudio/dom-exceptions.html b/LayoutTests/webaudio/dom-exceptions.html
index 8cd26a476fd158954c7f53788ca51e0d404017e2..187b5b70f8491601e4ba75f2cd1c892a39b18c70 100644
--- a/LayoutTests/webaudio/dom-exceptions.html
+++ b/LayoutTests/webaudio/dom-exceptions.html
@@ -165,6 +165,13 @@ function runTest() {
     node.connect(context.destination);
     shouldThrow("node.gain.exponentialRampToValueAtTime(-1, 0.1)");
     shouldThrow("node.gain.exponentialRampToValueAtTime(0, 0.1)");
+
+    // Convolver buffer rate must match context rate. Create on offline context so we
+    // specify the context rate exactly, in case the test is run on platforms with different
+    // HW sample rates.
+    shouldNotThrow("oc = new webkitOfflineAudioContext(1, 44100, 44100)");
+    conv = oc.createConvolver();

[Ken Russell (switch to Gerrit), 2014/03/17 18:21:38]

Maybe this should be wrapped in a shouldNotThrow too for clarity.

[Raymond Toy, 2014/03/17 19:42:13]

Done.

Do you think this should be done in general in this file? (Not for this CL, of
course.)

[Ken Russell (switch to Gerrit), 2014/03/17 21:35:30]

If you think it will improve clarity of the test, then yes.

+    shouldThrow("conv.buffer = oc.createBuffer(1, 100, 22050)");
 }
 
 runTest();