media::WebAudioSourceProviderImpl, add support for getting a copy of passing AudioBuses

media/blink/webaudiosourceprovider_impl.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/blink/webaudiosourceprovider_impl.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 29 matching lines @@
   }
 
  private:
   base::Lock& lock_;
   const bool acquired_;
   DISALLOW_COPY_AND_ASSIGN(AutoTryLock);
 };
 
 }  // namespace
 
+// TeeFilter is a RenderCallback implementation that allows for a client to get
+// a copy of the data being rendered by the |renderer_| on Render(). This class
+// also holds to the necessary audio parameters.
+class WebAudioSourceProviderImpl::TeeFilter
+    : public AudioRendererSink::RenderCallback {
+ public:
+  TeeFilter(AudioRendererSink::RenderCallback* const renderer,

[DaleCurtis, 2016/03/12 03:24:42]

RenderCallback* const doesn't mean anything as a parameter. I.e drop const.

[mcasas, 2016/03/16 18:50:50]

Done.

+            int channels,
+            int sample_rate)
+      : renderer_(renderer), channels_(channels), sample_rate_(sample_rate) {
+    DCHECK(renderer_);
+  }
+  ~TeeFilter() override {}
+
+  // AudioRendererSink::RenderCallback implementation.
+  // These are forwarders to |renderer_| and are here to allow for a client to
+  // get a copy of the rendered audio by RegisterCopyAudioCallback().
+  int Render(AudioBus* audio_bus,
+             uint32_t delay_milliseconds,
+             uint32_t frames_skipped) override;
+  void OnRenderError() override;
+
+  int channels() const { return channels_; }
+  int sample_rate() const { return sample_rate_; }
+  void set_copy_audio_bus_callback(
+      const WebAudioSourceProviderImpl::CopyAudioCB& callback) {
+    copy_audio_bus_callback_ = callback;
+  }
+
+ private:
+  AudioRendererSink::RenderCallback* const renderer_;
+  const int channels_;
+  const int sample_rate_;
+
+  WebAudioSourceProviderImpl::CopyAudioCB copy_audio_bus_callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(TeeFilter);
+};
+
 WebAudioSourceProviderImpl::WebAudioSourceProviderImpl(
     const scoped_refptr<RestartableAudioRendererSink>& sink)
-    : channels_(0),
-      sample_rate_(0),
-      volume_(1.0),
+    : volume_(1.0),
       state_(kStopped),
-      renderer_(NULL),
-      client_(NULL),
+      client_(nullptr),
       sink_(sink),
       weak_factory_(this) {}
 
 WebAudioSourceProviderImpl::~WebAudioSourceProviderImpl() {
 }
 
 void WebAudioSourceProviderImpl::setClient(
     blink::WebAudioSourceProviderClient* client) {
   base::AutoLock auto_lock(sink_lock_);
   if (client && client != client_) {
     // Detach the audio renderer from normal playback.
     sink_->Stop();
 
     // The client will now take control by calling provideInput() periodically.
     client_ = client;
 
     set_format_cb_ = BindToCurrentLoop(base::Bind(
         &WebAudioSourceProviderImpl::OnSetFormat, weak_factory_.GetWeakPtr()));
 
-    // If |renderer_| is set, then run |set_format_cb_| to send |client_|
-    // the current format info. If |renderer_| is not set, then |set_format_cb_|
-    // will get called when Initialize() is called.
-    // Note: Always using |set_format_cb_| ensures we have the same
-    // locking order when calling into |client_|.
-    if (renderer_)
+    // If |tee_filter_| is set, it means we have been Initialize()d - then run
+    // |set_format_cb_| to send |client_| the current format info. Otherwise
+    // |set_format_cb_| will get called when Initialize() is called.
+    // Note: Always using |set_format_cb_| ensures we have the same locking
+    // order when calling into |client_|.
+    if (tee_filter_)
       base::ResetAndReturn(&set_format_cb_).Run();
   } else if (!client && client_) {
     // Restore normal playback.
-    client_ = NULL;
+    client_ = nullptr;
     sink_->SetVolume(volume_);
     if (state_ >= kStarted)
       sink_->Start();
     if (state_ >= kPlaying)
       sink_->Play();
   }
 }
 
 void WebAudioSourceProviderImpl::provideInput(
     const WebVector<float*>& audio_data, size_t number_of_frames) {
   if (!bus_wrapper_ ||
       static_cast<size_t>(bus_wrapper_->channels()) != audio_data.size()) {
     bus_wrapper_ = AudioBus::CreateWrapper(static_cast<int>(audio_data.size()));
   }
 
-  bus_wrapper_->set_frames(static_cast<int>(number_of_frames));
+  const int incoming_number_of_frames = static_cast<int>(number_of_frames);
+  bus_wrapper_->set_frames(incoming_number_of_frames);
   for (size_t i = 0; i < audio_data.size(); ++i)
     bus_wrapper_->SetChannelData(static_cast<int>(i), audio_data[i]);
 
   // Use a try lock to avoid contention in the real-time audio thread.
   AutoTryLock auto_try_lock(sink_lock_);
   if (!auto_try_lock.locked() || state_ != kPlaying) {
     // Provide silence if we failed to acquire the lock or the source is not
     // running.
     bus_wrapper_->Zero();
     return;
   }
 
-  DCHECK(renderer_);
   DCHECK(client_);
-  DCHECK_EQ(channels_, bus_wrapper_->channels());
-  const int frames = renderer_->Render(bus_wrapper_.get(), 0, 0);
-  if (frames < static_cast<int>(number_of_frames)) {
-    bus_wrapper_->ZeroFramesPartial(
-        frames,
-        static_cast<int>(number_of_frames - frames));
-  }
+  DCHECK(tee_filter_);
+  DCHECK_EQ(tee_filter_->channels(), bus_wrapper_->channels());
+  const int frames = tee_filter_->Render(bus_wrapper_.get(), 0, 0);
+  if (frames < incoming_number_of_frames)
+    bus_wrapper_->ZeroFramesPartial(frames, incoming_number_of_frames - frames);
 
   bus_wrapper_->Scale(volume_);
 }
 
 void WebAudioSourceProviderImpl::Start() {
   base::AutoLock auto_lock(sink_lock_);
-  DCHECK(renderer_);
+  DCHECK(tee_filter_);
   DCHECK_EQ(state_, kStopped);
   state_ = kStarted;
   if (!client_)
     sink_->Start();
 }
 
 void WebAudioSourceProviderImpl::Stop() {
   base::AutoLock auto_lock(sink_lock_);
   state_ = kStopped;
   if (!client_)
@@ skipping 22 matching lines @@
   if (!client_)
     sink_->SetVolume(volume);
   return true;
 }
 
 OutputDevice* WebAudioSourceProviderImpl::GetOutputDevice() {
   base::AutoLock auto_lock(sink_lock_);
   return sink_->GetOutputDevice();
 }
 
-void WebAudioSourceProviderImpl::Initialize(
-    const AudioParameters& params,
-    RenderCallback* renderer) {
+void WebAudioSourceProviderImpl::Initialize(const AudioParameters& params,
+                                            RenderCallback* renderer) {
   base::AutoLock auto_lock(sink_lock_);
-  renderer_ = renderer;
+  DCHECK_EQ(state_, kStopped);
 
-  DCHECK_EQ(state_, kStopped);
-  sink_->Initialize(params, renderer);
+  tee_filter_ = make_scoped_ptr(
+      new TeeFilter(renderer, params.channels(), params.sample_rate()));
 
-  // Keep track of the format in case the client hasn't yet been set.
-  channels_ = params.channels();
-  sample_rate_ = params.sample_rate();
+  sink_->Initialize(params, tee_filter_.get());
 
   if (!set_format_cb_.is_null())
     base::ResetAndReturn(&set_format_cb_).Run();
 }
 
+void WebAudioSourceProviderImpl::RegisterCopyAudioCallback(
+    const CopyAudioCB& callback) {
+  DCHECK(!callback.is_null());
+  DCHECK(tee_filter_);
+  tee_filter_->set_copy_audio_bus_callback(callback);
+}
+
+void WebAudioSourceProviderImpl::ClearCopyAudioCallback() {
+  DCHECK(tee_filter_);
+  tee_filter_->set_copy_audio_bus_callback(CopyAudioCB());
+}
+
 void WebAudioSourceProviderImpl::OnSetFormat() {
   base::AutoLock auto_lock(sink_lock_);
   if (!client_)
     return;
 
   // Inform Blink about the audio stream format.
-  client_->setFormat(channels_, sample_rate_);
+  client_->setFormat(tee_filter_->channels(), tee_filter_->sample_rate());
+}
+
+int WebAudioSourceProviderImpl::RenderForTesting(AudioBus* audio_bus) {
+  return tee_filter_->Render(audio_bus, 0, 0);
+}
+
+int WebAudioSourceProviderImpl::TeeFilter::Render(AudioBus* audio_bus,
+                                                  uint32_t delay_milliseconds,
+                                                  uint32_t frames_skipped) {
+  const int num_rendered_frames =
+      renderer_->Render(audio_bus, delay_milliseconds, frames_skipped);
+
+  if (!copy_audio_bus_callback_.is_null()) {
+    scoped_ptr<AudioBus> bus_copy =
+        AudioBus::Create(audio_bus->channels(), audio_bus->frames());
+    audio_bus->CopyTo(bus_copy.get());
+    copy_audio_bus_callback_.Run(std::move(bus_copy), delay_milliseconds,
+                                 sample_rate_);
+  }
+
+  return num_rendered_frames;
+}
+
+void WebAudioSourceProviderImpl::TeeFilter::OnRenderError() {
+  renderer_->OnRenderError();
 }
 
 }  // namespace media