Remove input/source components from AudioDestination

third_party/WebKit/Source/platform/audio/AudioDestination.cpp

Index: third_party/WebKit/Source/platform/audio/AudioDestination.cpp
diff --git a/third_party/WebKit/Source/platform/audio/AudioDestination.cpp b/third_party/WebKit/Source/platform/audio/AudioDestination.cpp
index 448ef74fd961e414e1e6d2687c52e6a08d794a85..72444184e89d65df7c8a29bf9fb77efc2bc4af9b 100644
--- a/third_party/WebKit/Source/platform/audio/AudioDestination.cpp
+++ b/third_party/WebKit/Source/platform/audio/AudioDestination.cpp
@@ -64,8 +64,6 @@ AudioDestination::AudioDestination(AudioIOCallback& callback,
                                    PassRefPtr<SecurityOrigin> securityOrigin)
     : m_callback(callback),
       m_numberOfOutputChannels(numberOfOutputChannels),
-      m_inputBus(AudioBus::create(numberOfInputChannels,
-                                  AudioUtilities::kRenderQuantumFrames)),
       m_renderBus(AudioBus::create(numberOfOutputChannels,
                                    AudioUtilities::kRenderQuantumFrames,
                                    false)),
@@ -130,19 +128,6 @@ AudioDestination::AudioDestination(AudioIOCallback& callback,
   m_fifo =
       WTF::wrapUnique(new AudioPullFIFO(*this, numberOfOutputChannels, fifoSize,
                                         AudioUtilities::kRenderQuantumFrames));
-
-  // Input buffering.
-  m_inputFifo = WTF::makeUnique<AudioFIFO>(numberOfInputChannels, fifoSize);
-
-  // If the callback size does not match the render size, then we need to
-  // buffer some extra silence for the input. Otherwise, we can over-consume
-  // the input FIFO.
-  if (m_callbackBufferSize != AudioUtilities::kRenderQuantumFrames) {
-    // FIXME: handle multi-channel input and don't hard-code to stereo.
-    RefPtr<AudioBus> silence =
-        AudioBus::create(2, AudioUtilities::kRenderQuantumFrames);
-    m_inputFifo->push(silence.get());
-  }
 }
 
 AudioDestination::~AudioDestination() {
@@ -171,8 +156,7 @@ unsigned long AudioDestination::maxChannelCount() {
   return static_cast<float>(Platform::current()->audioHardwareOutputChannels());
 }
 
-void AudioDestination::render(const WebVector<float*>& sourceData,
-                              const WebVector<float*>& audioData,
+void AudioDestination::render(const WebVector<float*>& audioData,
                               size_t numberOfFrames,
                               double delay,
                               double delayTimestamp,
@@ -196,15 +180,6 @@ void AudioDestination::render(const WebVector<float*>& sourceData,
   m_outputPosition.timestamp = delayTimestamp;
   m_outputPositionReceivedTimestamp = base::TimeTicks::Now();
 
-  // Buffer optional live input.
-  if (sourceData.size() >= 2) {
-    // FIXME: handle multi-channel input and don't hard-code to stereo.
-    RefPtr<AudioBus> wrapperBus = AudioBus::create(2, numberOfFrames, false);
-    wrapperBus->setChannelMemory(0, sourceData[0], numberOfFrames);
-    wrapperBus->setChannelMemory(1, sourceData[1], numberOfFrames);
-    m_inputFifo->push(wrapperBus.get());
-  }
-
   for (unsigned i = 0; i < m_numberOfOutputChannels; ++i)
     m_renderBus->setChannelMemory(i, audioData[i], numberOfFrames);
 
@@ -214,12 +189,6 @@ void AudioDestination::render(const WebVector<float*>& sourceData,
 }
 
 void AudioDestination::provideInput(AudioBus* bus, size_t framesToProcess) {
-  AudioBus* sourceBus = nullptr;
-  if (m_inputFifo->framesInFifo() >= framesToProcess) {
-    m_inputFifo->consume(m_inputBus.get(), framesToProcess);
-    sourceBus = m_inputBus.get();
-  }
-
   AudioIOPosition outputPosition = m_outputPosition;
 
   // If platfrom buffer is more than two times longer than |framesToProcess|
@@ -237,7 +206,7 @@ void AudioDestination::provideInput(AudioBus* bus, size_t framesToProcess) {
   if (outputPosition.position < 0.0)
     outputPosition.position = 0.0;
 
-  m_callback.render(sourceBus, bus, framesToProcess, outputPosition);
+  m_callback.render(nullptr, bus, framesToProcess, outputPosition);

[Raymond Toy, 2016/12/20 21:46:49]

Can this be updated to remove the first arg?

 }
 
 }  // namespace blink