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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: LayoutTests/webaudio/audionode-disconnect.html

Issue 886173004: Fix AudioNode.disconnect() to support selective disconnection. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fixed and added layout test parts. Created 5 years, 10 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
« no previous file with comments | « no previous file | LayoutTests/webaudio/audionode-disconnect-audioparam.html » ('j') | LayoutTests/webaudio/audionode-disconnect-audioparam.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/webaudio/audionode-disconnect.html
diff --git a/LayoutTests/webaudio/audionode-disconnect.html b/LayoutTests/webaudio/audionode-disconnect.html
new file mode 100644
index 0000000000000000000000000000000000000000..1e81ec05802430072a6243e0e4d638235e57ab46
--- /dev/null
+++ b/LayoutTests/webaudio/audionode-disconnect.html
@@ -0,0 +1,281 @@
+<!DOCTYPE html>
+<html>
+
+<head>
+ <script src="../resources/js-test.js"></script>
+ <script src="resources/compatibility.js"></script>
+ <script src="resources/audio-testing.js"></script>
+</head>
+
+<body>
+ <script>
+ description('Test disconnect() method on AudioNode destination.');
+ window.jsTestIsAsync = true;
+
+ var audit = Audit.createTaskRunner();
+
+ // Task 1: test disconnect() method.
+ audit.defineTask('disconnect()', function (done) {
+
+ // Connect a source to multiple gain nodes, each connected to the
+ // destination. Then disconnect the source. The expected output should be
+ // all zeros since the source was disconnected.
+ var context = new OfflineAudioContext(1, 128, 44100);
+ var source = context.createBufferSource();
+ var buffer1ch = createTestingAudioBuffer(context, 1, 128);
+ var gain1 = context.createGain();
+ var gain2 = context.createGain();
+ var gain3 = context.createGain();
+
+ source.buffer = buffer1ch;
+
+ source.connect(gain1);
+ source.connect(gain2);
+ source.connect(gain3);
+ gain1.connect(context.destination);
+ gain2.connect(context.destination);
+ gain3.connect(context.destination);
+ source.start();
+
+ // This disconnects everything.
+ source.disconnect();
+
+ context.startRendering().then(function (buffer) {
+ var channelData = buffer.getChannelData(0);
+
+ // With everything disconnected, the result should be zero.
+ Should.haveValueInChannel(0, channelData);
+
+ }).then(done);
+ });
+
+ // Task 2: test disconnect(output) method.
+ audit.defineTask('disconnect(output)', function (done) {
+
+ // Create multiple connections from each output of a ChannelSplitter
+ // to a gain node. Then test if disconnecting a single output of splitter
+ // is actually disconnected.
+ var context = new OfflineAudioContext(1, 128, 44100);
+ var source = context.createBufferSource();
+ var buffer3ch = createTestingAudioBuffer(context, 3, 128);
+ var splitter = context.createChannelSplitter(3);
+ var sum = context.createGain();
+
+ source.buffer = buffer3ch;
+
+ source.connect(splitter);
+ splitter.connect(sum, 0);
+ splitter.connect(sum, 1);
+ splitter.connect(sum, 2);
+ sum.connect(context.destination);
+ source.start();
+
+ // This disconnects the second output.
+ splitter.disconnect(1);
+
+ context.startRendering().then(function (buffer) {
+ var channelData = buffer.getChannelData(0);
+
+ // The rendered channel should contain 4. (= 1 + 0 + 3)
+ Should.haveValueInChannel(4, channelData);
+
+ }).then(done);
+ });
+
+ // Task 3: test disconnect(AudioNode) method.
+ audit.defineTask('disconnect(AudioNode)', function (done) {
+
+ // Connect a source to multiple gain nodes. Then test if disconnecting a
+ // single destination selectively works correctly.
+ var context = new OfflineAudioContext(1, 128, 44100);
+ var source = context.createBufferSource();
+ var buffer1ch = createTestingAudioBuffer(context, 1, 128);
+ var gain1 = context.createGain();
+ var gain2 = context.createGain();
+ var gain3 = context.createGain();
+ var orphan = context.createGain();
+
+ source.buffer = buffer1ch;
+
+ source.connect(gain1);
+ source.connect(gain2);
+ source.connect(gain3);
+ gain1.connect(context.destination);
+ gain2.connect(context.destination);
+ gain3.connect(context.destination);
+ source.start();
+
+ source.disconnect(gain2);
+
+ context.startRendering().then(function (buffer) {
+ var channelData = buffer.getChannelData(0);
+
+ // The |sum| gain node should produce value 2. (1 + 0 + 1 = 2)
+ Should.haveValueInChannel(2, channelData);
+
+ }).then(done);
+ });
+
+ // Task 4: test disconnect(AudioNode, output) method.
+ audit.defineTask('disconnect(AudioNode, output)', function (done) {
+
+ // Connect a 2-channel buffer with [1, 2] in each channel to a
Raymond Toy 2015/02/13 17:08:16 "[1, 2] in each channel" is a bit confusing. This
hongchan 2015/02/13 20:19:26 Yes, that's what I meant. I will rephrase it.
+ // ChannelSplitter, then connect the splitter to 2 gain nodes as shown
+ // below:
+ // (1) splitter#0 => gain1
+ // (2) splitter#0 => gain2
+ // (3) splitter#1 => gain2
+ // Then disconnect (2) and verify if the selective disconnection on a
+ // specified output of the destination node works correctly.
+ var context = new OfflineAudioContext(1, 128, 44100);
+ var source = context.createBufferSource();
+ var buffer2ch = createTestingAudioBuffer(context, 2, 128);
+ var splitter = context.createChannelSplitter(2);
+ var gain1 = context.createGain();
+ var gain2 = context.createGain();
+
+ source.buffer = buffer2ch;
+
+ source.connect(splitter);
+ splitter.connect(gain1, 0); // gain1 gets [1]
+ splitter.connect(gain2, 0); // gain2 gets [1 + 2]
Raymond Toy 2015/02/13 17:08:16 I think both of these would be clearer if you just
hongchan 2015/02/13 20:19:26 Done.
+ splitter.connect(gain2, 1);
+ gain1.connect(context.destination);
+ gain2.connect(context.destination);
+ source.start();
+
+ splitter.disconnect(gain2, 0); // Now gain2 gets [2]
+
+ context.startRendering().then(function (buffer) {
+ var channelData0 = buffer.getChannelData(0);
+
+ // The sum of gain1 and gain2 should produce value 3. (= 1 + 2)
+ Should.haveValueInChannel(3, channelData0);
+
+ }).then(done);
+ });
+
+ // Task 5: test disconnect(AudioNode, output, input) method.
+ audit.defineTask('disconnect(AudioNode, output, input)', function (done) {
+
+ // Create a 3-channel buffer with [1, 2, 3] in each channel and then pass
+ // it through a splitter and a merger. Each input/output of the splitter
+ // and the merger is connected in a sequential order as shown below.
+ // (1) splitter#0 => merger#0
+ // (2) splitter#1 => merger#1
+ // (3) splitter#2 => merger#2
+ // Then disconnect (3) and verify if each channel contains [1] and [2]
+ // respectively.
+ var context = new OfflineAudioContext(3, 128, 44100);
+ var source = context.createBufferSource();
+ var buffer3ch = createTestingAudioBuffer(context, 3, 128);
+ var splitter = context.createChannelSplitter(3);
+ var merger = context.createChannelMerger(3);
+
+ source.buffer = buffer3ch;
+
+ source.connect(splitter);
+ splitter.connect(merger, 0, 0);
+ splitter.connect(merger, 1, 1);
+ splitter.connect(merger, 2, 2);
+ merger.connect(context.destination);
+ source.start();
+
+ splitter.disconnect(merger, 2, 2);
+
+ context.startRendering().then(function (buffer) {
+ var channelData0 = buffer.getChannelData(0);
+ var channelData1 = buffer.getChannelData(1);
+ var channelData2 = buffer.getChannelData(2);
+
+ // Each channel should have 1, 2, and 0 respectively.
+ Should.haveValueInChannel(1, channelData0);
+ Should.haveValueInChannel(2, channelData1);
+ Should.haveValueInChannel(0, channelData2);
+
+ }).then(done);
+ });
+
+ // Task 6: exception checks.
+ audit.defineTask('exceptions', function (done) {
+ var context = new AudioContext();
Raymond Toy 2015/02/13 17:08:16 Is AudioContext required or can OfflineAudioContex
hongchan 2015/02/13 20:19:26 Fixed.
+ var gain1 = context.createGain();
+ var splitter = context.createChannelSplitter(2);
+ var merger = context.createChannelMerger(2);
+ var gain2 = context.createGain();
+ var gain3 = context.createGain();
+
+ gain1.connect(splitter);
+ splitter.connect(gain2, 0);
+ splitter.connect(gain3, 1);
+ splitter.connect(merger, 0, 0);
+ splitter.connect(merger, 1, 1);
+ gain2.connect(gain3);
+ gain3.connect(context.destination);
+ merger.connect(context.destination);
Raymond Toy 2015/02/13 17:08:16 Add a brief description of why you're connecting t
+
+ // There is no output #2. An exception should be thrown.
+ Should.throwWithType('IndexSizeError', function () {
+ splitter.disconnect(2);
+ });
+
+ // Disconnecting the output already disconnected should not throw.
+ Should.notThrow(function () {
+ splitter.disconnect(1);
+ splitter.disconnect(1);
+ });
+
+ // gain2 is already disconnected. An exception should be thrown.
+ Should.throwWithType('InvalidAccessError', function () {
+ gain1.disconnect(gain2);
Raymond Toy 2015/02/13 17:08:16 How is gain1 connected to gain2? I think you're tr
hongchan 2015/02/13 20:19:26 Oops. This is a mistake.
+ gain1.disconnect(gain2);
+ });
+
+ // gain1 and gain3 are not connected. An exception should be thrown.
+ Should.throwWithType('InvalidAccessError', function () {
+ gain1.disconnect(gain3);
+ });
+
+ // There is no output #2 in the splitter. An exception should be thrown.
+ Should.throwWithType('IndexSizeError', function () {
+ splitter.disconnect(gain2, 2);
+ });
+
+ // splitter and gain1 are not connected. An exception should be thrown.
+ Should.throwWithType('InvalidAccessError', function () {
+ splitter.disconnect(gain1, 0);
+ });
+
+ // splitter and gain3 are not connection. An exception should be thrown.
Raymond Toy 2015/02/13 17:08:16 "connection" -> "connected". Plus this comment do
hongchan 2015/02/13 20:19:26 Fixed.
+ Should.throwWithType('InvalidAccessError', function () {
+ splitter.disconnect(gain3, 0, 0);
+ });
+
+ // The output index is out of bound. An exception should be thrown.
+ Should.throwWithType('IndexSizeError', function () {
+ splitter.disconnect(merger, 3, 0);
+ });
+
+ done();
+ });
+
+ audit.defineTask('finish', function (done) {
+ finishJSTest();
+ done();
+ });
+
+ audit.runTasks(
+ 'disconnect()',
+ 'disconnect(output)',
+ 'disconnect(AudioNode)',
+ 'disconnect(AudioNode, output)',
+ 'disconnect(AudioNode, output, input)',
+ 'exceptions',
+ 'finish'
+ );
+
+ successfullyParsed = true;
+ </script>
+</body>
+
+</html>
« no previous file with comments | « no previous file | LayoutTests/webaudio/audionode-disconnect-audioparam.html » ('j') | LayoutTests/webaudio/audionode-disconnect-audioparam.html » ('J')

Powered by Google App Engine
This is Rietveld 408576698