Tab Audio Capture: Browser-side connect/disconnect functionality.

content/browser/renderer_host/media/audio_renderer_host.cc

 // Copyright (c) 2012 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 "content/browser/renderer_host/media/audio_renderer_host.h"
 
 #include "base/bind.h"
 #include "base/metrics/histogram.h"
 #include "base/process.h"
 #include "base/shared_memory.h"
 #include "content/browser/browser_main_loop.h"
+#include "content/browser/renderer_host/media/audio_mirroring_manager.h"
 #include "content/browser/renderer_host/media/audio_sync_reader.h"
 #include "content/common/media/audio_messages.h"
 #include "content/public/browser/media_observer.h"
 #include "media/audio/shared_memory_util.h"
 #include "media/base/audio_bus.h"
 #include "media/base/limits.h"
 
 using media::AudioBus;
 
 namespace content {
 
+namespace {
+
+// Simple wrapper around a media::AudioOutputController to implement the
+// AudioMirroringManager::Diverter interface.
+class ControllerDiverter : public AudioMirroringManager::Diverter {

[DaleCurtis, 2013/01/02 21:56:44]

ISTM that you could just have AudioOutputController implement this interface and
forgo this wrapper class.

[miu, 2013/01/03 22:30:42]

Done.

I had thought about this, but I quickly dismissed the idea because of component
boundaries (the AMM::Diverter interface is in content, whereas AOC is in media).
 However, I just tried moving the interface definition into media (i.e.,
AudioMirroringManager::Diverter is now AudioOutputStream::AudioSourceDiverter),
and it seems to work well w.r.t. code organization.

Do you like it better this way?  I could see arguments for doing things either
way.

+ public:
+  void SetController(media::AudioOutputController* controller);
+
+  // AudioMirroringManager::Diverter implementation.
+  virtual const media::AudioParameters& GetAudioParameters() OVERRIDE;
+  virtual void StartDiverting(media::AudioOutputStream* stream) OVERRIDE;
+  virtual void StopDiverting() OVERRIDE;
+
+ private:
+  media::AudioOutputController* controller_;  // Not owned.
+};

[DaleCurtis, 2013/01/02 21:56:44]

Disallow copy and assign? Why not make provide controller_ in the constructor
instead of having a SetController method?  Just make the AudioEntry version a
scoped_ptr.

[miu, 2013/01/03 22:30:42]

No longer applicable.

+
+void ControllerDiverter::SetController(media::AudioOutputController* ctlr) {
+  controller_ = ctlr;

[DaleCurtis, 2013/01/02 21:56:44]

DCHECK(ctlr) for haystack hunts.

[miu, 2013/01/03 22:30:42]

No longer applicable.

+}
+
+const media::AudioParameters& ControllerDiverter::GetAudioParameters() {
+  return controller_->params();
+}
+
+void ControllerDiverter::StartDiverting(media::AudioOutputStream* stream) {
+  controller_->StartDiverting(stream);
+}
+
+void ControllerDiverter::StopDiverting() {
+  controller_->StopDiverting();
+}
+
+}  // namespace
+
 struct AudioRendererHost::AudioEntry {
   AudioEntry();
   ~AudioEntry();
 
   // The AudioOutputController that manages the audio stream.
   scoped_refptr<media::AudioOutputController> controller;
+  ControllerDiverter diverter;
 
   // The audio stream ID.
   int stream_id;
 
+  // The routing ID of the source render view.
+  int render_view_id;
+
   // Shared memory for transmission of the audio data.
   base::SharedMemory shared_memory;
 
   // The synchronous reader to be used by the controller. We have the
   // ownership of the reader.
   scoped_ptr<media::AudioOutputController::SyncReader> reader;
 
   // Set to true after we called Close() for the controller.
   bool pending_close;
 };
 
 AudioRendererHost::AudioEntry::AudioEntry()
     : stream_id(0),
+      render_view_id(MSG_ROUTING_NONE),
       pending_close(false) {
 }
 
 AudioRendererHost::AudioEntry::~AudioEntry() {}
 
 ///////////////////////////////////////////////////////////////////////////////
 // AudioRendererHost implementations.
 AudioRendererHost::AudioRendererHost(
-    media::AudioManager* audio_manager, MediaObserver* media_observer)
-    : audio_manager_(audio_manager),
+    int render_process_id,
+    media::AudioManager* audio_manager,
+    AudioMirroringManager* mirroring_manager,
+    MediaObserver* media_observer)
+    : render_process_id_(render_process_id),
+      audio_manager_(audio_manager),
+      mirroring_manager_(mirroring_manager),
       media_observer_(media_observer) {
 }
 
 AudioRendererHost::~AudioRendererHost() {
   DCHECK(audio_entries_.empty());
 }
 
 void AudioRendererHost::OnChannelClosing() {
   BrowserMessageFilter::OnChannelClosing();
 
@@ skipping 196 matching lines @@
   if (!reader->Init()) {
     SendErrorMessage(stream_id);
     return;
   }
 
   // If we have successfully created the SyncReader then assign it to the
   // entry and construct an AudioOutputController.
   entry->reader.reset(reader.release());
   entry->controller = media::AudioOutputController::Create(
       audio_manager_, this, audio_params, entry->reader.get());
-
   if (!entry->controller) {
     SendErrorMessage(stream_id);
     return;
   }
 
+  entry->diverter.SetController(entry->controller);
+
   // If we have created the controller successfully, create an entry and add it
   // to the map.
   entry->stream_id = stream_id;
   audio_entries_.insert(std::make_pair(stream_id, entry.release()));
   if (media_observer_)
     media_observer_->OnSetAudioStreamStatus(this, stream_id, "created");
 }
 
 void AudioRendererHost::OnAssociateStreamWithProducer(int stream_id,
                                                       int render_view_id) {
-  // TODO(miu): Will use render_view_id in upcoming change.
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+
   DVLOG(1) << "AudioRendererHost@" << this
            << "::OnAssociateStreamWithProducer(stream_id=" << stream_id
            << ", render_view_id=" << render_view_id << ")";
+
+  AudioEntry* const entry = LookupById(stream_id);
+  if (!entry) {
+    SendErrorMessage(stream_id);
+    return;
+  }
+
+  if (entry->render_view_id == render_view_id)
+    return;  // No change.
+
+  if (mirroring_manager_) {

[DaleCurtis, 2013/01/02 21:56:44]

Mirroring manager should always be present  since it's provided during the
constructor? Just DCHECK() in constructor?

[miu, 2013/01/03 22:30:42]

Done.  And fixed things so MediaObserver must never be NULL either.

+    mirroring_manager_->RemoveDiverter(
+        render_process_id_, entry->render_view_id, &entry->diverter);
+  }
+  entry->render_view_id = render_view_id;
+  if (mirroring_manager_) {
+    mirroring_manager_->AddDiverter(
+        render_process_id_, entry->render_view_id, &entry->diverter);
+  }
 }
 
 void AudioRendererHost::OnPlayStream(int stream_id) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   AudioEntry* entry = LookupById(stream_id);
   if (!entry) {
     SendErrorMessage(stream_id);
     return;
   }
@@ skipping 71 matching lines @@
   for (AudioEntryMap::iterator i = audio_entries_.begin();
        i != audio_entries_.end(); ++i) {
     CloseAndDeleteStream(i->second);
   }
 }
 
 void AudioRendererHost::CloseAndDeleteStream(AudioEntry* entry) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   if (!entry->pending_close) {
+    if (mirroring_manager_) {

[DaleCurtis, 2013/01/02 21:56:44]

Ditto.

[miu, 2013/01/03 22:30:42]

Done.

+      mirroring_manager_->RemoveDiverter(
+          render_process_id_, entry->render_view_id, &entry->diverter);
+    }
     entry->controller->Close(
         base::Bind(&AudioRendererHost::DeleteEntry, this, entry));
     entry->pending_close = true;
   }
 }
 
 void AudioRendererHost::DeleteEntry(AudioEntry* entry) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   // Delete the entry when this method goes out of scope.
@@ skipping 42 matching lines @@
   return NULL;
 }
 
 media::AudioOutputController* AudioRendererHost::LookupControllerByIdForTesting(
     int stream_id) {
   AudioEntry* const entry = LookupById(stream_id);
   return entry ? entry->controller : NULL;
 }
 
 }  // namespace content