Mojo interfaces needed for switching audio rendering stream creation and closing from IPC to Mojo

content/renderer/media/audio_message_filter.cc

Index: content/renderer/media/audio_message_filter.cc
diff --git a/content/renderer/media/audio_message_filter.cc b/content/renderer/media/audio_message_filter.cc
index 9e7322d128e65f0d5f9507a544c8cf01943e7d16..c25564b0cfedf2cd993883e8df6ef24badb3b619 100644
--- a/content/renderer/media/audio_message_filter.cc
+++ b/content/renderer/media/audio_message_filter.cc
@@ -10,6 +10,7 @@
 #include "base/single_thread_task_runner.h"
 #include "base/strings/stringprintf.h"
 #include "content/common/media/audio_messages.h"
+#include "content/renderer/media/audio_output_client.h"
 #include "content/renderer/media/webrtc_logging.h"
 #include "content/renderer/render_thread_impl.h"
 #include "ipc/ipc_logging.h"
@@ -106,8 +107,8 @@ void AudioMessageFilter::AudioOutputIPCImpl::CreateStream(
   if (stream_id_ == kStreamIDNotSet)
     stream_id_ = filter_->delegates_.Add(delegate);
 
-  filter_->Send(
-      new AudioHostMsg_CreateStream(stream_id_, render_frame_id_, params));
+  filter_->audio_output_client_->CreateStream(stream_id_, render_frame_id_,

[Henrik Grunell, 2016/04/19 15:36:09]

Is it possible to not go through the IPCImpl in this intermediate state this CL
creates? It's not a big deal since upcoming CLs will remove the filter, but it
would be nice. Add todo comments that this is temporary and will soon be
changed.

[rchtara, 2016/04/21 09:10:18]

I think that this is going to be very hard, as many functions still rely on
delegates_ map to detect errors.
 When all the IPC is removed, we could have a similar map in AudioOutputClient
and use it to detect problems. 
Right now, the ipc part is still using the delegates_ map, the mojo part needs
to use it too.
So, i think it'd going to be better to be patient and replace these functions
when pause,play.. are mojofyied .

[Henrik Grunell, 2016/04/22 08:23:03]

Acknowledged.

+                                              params);
   stream_created_ = true;
 }
 
@@ -124,7 +125,7 @@ void AudioMessageFilter::AudioOutputIPCImpl::PauseStream() {
 void AudioMessageFilter::AudioOutputIPCImpl::CloseStream() {
   DCHECK(filter_->io_task_runner_->BelongsToCurrentThread());
   DCHECK_NE(stream_id_, kStreamIDNotSet);
-  filter_->Send(new AudioHostMsg_CloseStream(stream_id_));
+  filter_->audio_output_client_->CloseStream(stream_id_);
   filter_->delegates_.Remove(stream_id_);
   stream_id_ = kStreamIDNotSet;
   stream_created_ = false;
@@ -149,7 +150,6 @@ bool AudioMessageFilter::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(AudioMessageFilter, message)
     IPC_MESSAGE_HANDLER(AudioMsg_NotifyDeviceAuthorized, OnDeviceAuthorized)
-    IPC_MESSAGE_HANDLER(AudioMsg_NotifyStreamCreated, OnStreamCreated)
     IPC_MESSAGE_HANDLER(AudioMsg_NotifyStreamStateChanged, OnStreamStateChanged)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()