Defer changes to channelInterpretation to pre/post rendering.

third_party/WebKit/Source/modules/webaudio/AudioNode.cpp

 /*
  * Copyright (C) 2010, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 79 matching lines @@
 void AudioHandler::clearInternalStateWhenDisabled()
 {
 }
 
 void AudioHandler::dispose()
 {
     ASSERT(isMainThread());
     ASSERT(context()->isGraphOwner());
 
     context()->deferredTaskHandler().removeChangedChannelCountMode(this);
+    context()->deferredTaskHandler().removeChangedChannelInterpretation(this);
     context()->deferredTaskHandler().removeAutomaticPullNode(this);
     for (auto& output : m_outputs)
         output->dispose();
     m_node = nullptr;
 }
 
 AudioNode* AudioHandler::node() const
 {
     ASSERT(isMainThread());
     return m_node;
@@ skipping 163 matching lines @@
     }
     ASSERT_NOT_REACHED();
     return "";
 }
 
 void AudioHandler::setChannelInterpretation(const String& interpretation, ExceptionState& exceptionState)
 {
     ASSERT(isMainThread());
     AbstractAudioContext::AutoLocker locker(context());
 
+    AudioBus::ChannelInterpretation oldMode = m_channelInterpretation;
+
     if (interpretation == "speakers") {
-        m_channelInterpretation = AudioBus::Speakers;
+        m_newChannelInterpretation = AudioBus::Speakers;
     } else if (interpretation == "discrete") {
-        m_channelInterpretation = AudioBus::Discrete;
+        m_newChannelInterpretation = AudioBus::Discrete;
     } else {
         ASSERT_NOT_REACHED();
     }
+
+    if (m_newChannelInterpretation != oldMode)
+        context()->deferredTaskHandler().addChangedChannelInterpretation(this);
 }
 
 void AudioHandler::updateChannelsForInputs()
 {
     for (auto& input : m_inputs)
         input->changedOutputs();
 }
 
 void AudioHandler::processIfNecessary(size_t framesToProcess)
 {
@@ skipping 199 matching lines @@
 }
 
 #endif // DEBUG_AUDIONODE_REFERENCES
 
 void AudioHandler::updateChannelCountMode()
 {
     m_channelCountMode = m_newChannelCountMode;
     updateChannelsForInputs();
 }
 
+void AudioHandler::updateChannelInterpretation()
+{
+    m_channelInterpretation = m_newChannelInterpretation;
+}
+
 unsigned AudioHandler::numberOfOutputChannels() const
 {
     // This should only be called for ScriptProcessorNodes which are the only nodes where you can
     // have an output with 0 channels.  All other nodes have have at least one output channel, so
     // there's no reason other nodes should ever call this function.
     DCHECK(0) << "numberOfOutputChannels() not valid for node type " << getNodeType();
     return 1;
 }
 // ----------------------------------------------------------------
 
@@ skipping 414 matching lines @@
 void AudioNode::didAddOutput(unsigned numberOfOutputs)
 {
     m_connectedNodes.append(nullptr);
     ASSERT_UNUSED(numberOfOutputs, numberOfOutputs == m_connectedNodes.size());
     m_connectedParams.append(nullptr);
     ASSERT_UNUSED(numberOfOutputs, numberOfOutputs == m_connectedParams.size());
 }
 
 } // namespace blink