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..6ea8ca2f6ff0a67c4bb02d403d6fe2f62db8cbd5 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.
+    oc = new webkitOfflineAudioContext(1, 44100, 44100);

[Ken Russell (switch to Gerrit), 2014/03/14 23:54:12]

Possible to get this line into the debug output?

[Raymond Toy, 2014/03/17 18:09:19]

Added as a test so that the expected output contains this.

+    conv = context.createConvolver();

[Ken Russell (switch to Gerrit), 2014/03/14 23:54:12]

This isn't referencing "oc". Is it supposed to?

[Raymond Toy, 2014/03/17 18:09:19]

Yes, it should use oc. Fixed.

+    shouldThrow("conv.buffer = context.createBuffer(1, 100, 22050)");

[Ken Russell (switch to Gerrit), 2014/03/14 23:54:12]

Same here.

[Raymond Toy, 2014/03/17 18:09:19]

Done.

 }
 
 runTest();