Make reduction value of dynamics compressor zero when no sources are connected

Source/modules/webaudio/AudioNode.cpp

Index: Source/modules/webaudio/AudioNode.cpp
diff --git a/Source/modules/webaudio/AudioNode.cpp b/Source/modules/webaudio/AudioNode.cpp
index 8febafca9ef112741842e3825fe0ce89013884e3..9519ad52b010b81c87ac31de28fc2471e6e55d74 100644
--- a/Source/modules/webaudio/AudioNode.cpp
+++ b/Source/modules/webaudio/AudioNode.cpp
@@ -89,6 +89,10 @@ void AudioNode::uninitialize()
     m_isInitialized = false;
 }
 
+void AudioNode::clearInternalState()
+{
+}
+
 void AudioNode::dispose()
 {
     ASSERT(isMainThread());
@@ -475,6 +479,10 @@ void AudioNode::disableOutputsIfNecessary()
         // longer any active connections.
         if (nodeType() != NodeTypeConvolver && nodeType() != NodeTypeDelay) {
             m_isDisabled = true;
+            // Resets internal states for future external data request. (i.e. Gain reduction)

[Raymond Toy, 2014/10/21 20:47:57]

I think I would remove this comment.  The comment for the clearInternalState
method should explain what is happening and why.

[hongchan, 2014/10/21 20:58:53]

Acknowledged.

+            if (nodeType() == NodeTypeDynamicsCompressor) {

[Raymond Toy, 2014/10/21 20:47:57]

This if test isn't needed.  Every node now has a clearInternalState method, and
the DynamicsCompressorNode has the right implementation to clear the internal
state.

[hongchan, 2014/10/21 20:58:53]

I agree, but this was what we talked about it before so I left it there.
Removing.

+                clearInternalState();
+            }
             for (unsigned i = 0; i < m_outputs.size(); ++i)
                 output(i)->disable();
         }