1066063 - Graph shows no connections to any nodes after calling node.disconnect() on a node with several connections.

Status: RESOLVED INVALID

(DevTools Graveyard :: Web Audio Editor, defect)

Description

[Bat]

Attachment: webaudioeditor_nodegraph1_initial.png

Steps to reproduce:
* Open up "Web Audio Editor".
* Open up JavaScript console (split mode).
* Execute following code in the JavaScript console to create initial graph (attachment: 
webaudioeditor_nodegraph1_initial.png):

var context = new AudioContext();

var g1 = context.createGain();
var g2 = context.createGain();
var g3 = context.createGain();

//Connect all to destination
g1.connect(context.destination);
g2.connect(context.destination);
g3.connect(context.destination);

//Connect g1 to g2 and g3
g1.connect(g2);
g1.connect(g3);  

* Execute following code in JavaScript console:

//Disconnect g1 from g2
g1.disconnect(g2);


Result:
* By the graph shown, g1 is now disconnected from all other nodes. See attachment webaudioeditor_nodegraph2_result.png

Expected:
* By the graph g1 should remain connected to the AudioDestinationNode. See attachment webaudioeditor_nodegraph2_result.png where the expected connection is red line.

Comment 1

[Bat]

Attachment: webaudioeditor_nodegraph2_result.png

Comment 2

[Jordan Santell [:jsantell] [@jsantell]]

The `disconnect`[1] method in AudioNodes is for specifying which connection to disconnect via index; otherwise the node will disconnect all of it's outgoing connections

[1] http://webaudio.github.io/web-audio-api/#widl-AudioNode-disconnect-void

Comment 3

[Bat]

(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #2)
> The `disconnect`[1] method in AudioNodes is for specifying which connection
> to disconnect via index; otherwise the node will disconnect all of it's
> outgoing connections
> 
> [1] http://webaudio.github.io/web-audio-api/#widl-AudioNode-disconnect-void

https://developer.mozilla.org/en-US/docs/Web/API/AudioNode.disconnect
"The disconnect method of the AudioNode interface allows us to disconnect the current node from another one that it is already connected to.
...

Parameters
  Node
    The AudioNode you are disconnecting from.
"

Comment 4

[Jordan Santell [:jsantell] [@jsantell]]

I believe the MDN documentation is very incorrect -- Paul, is there some new disconnect API that specifies a specific node to disconnect?

Comment 5

[Paul Adenot (:padenot)]

Jordan, you're right, this simply does not exist. I'll go and fix the doc now.

Comment 6

[Bat]

Thanks for your time and effort.