IPC->mojo of audio_renderer_host

media/audio/audio_output_device.cc

Index: media/audio/audio_output_device.cc
diff --git a/media/audio/audio_output_device.cc b/media/audio/audio_output_device.cc
index 1d52911686edc6417d1c035e4406272c74596ab5..564ecd62819dde9b19a5603e91aa143bb28a9b56 100644
--- a/media/audio/audio_output_device.cc
+++ b/media/audio/audio_output_device.cc
@@ -278,7 +278,7 @@ void AudioOutputDevice::SetVolumeOnIOThread(double volume) {
     ipc_->SetVolume(volume);
 }
 
-void AudioOutputDevice::OnStateChanged(AudioOutputIPCDelegateState state) {
+void AudioOutputDevice::OnStateChanged(mojom::AudioOutputStreamState state) {
   DCHECK(task_runner()->BelongsToCurrentThread());
 
   // Do nothing if the stream has been closed.
@@ -288,10 +288,10 @@ void AudioOutputDevice::OnStateChanged(AudioOutputIPCDelegateState state) {
   // TODO(miu): Clean-up inconsistent and incomplete handling here.
   // http://crbug.com/180640
   switch (state) {
-    case AUDIO_OUTPUT_IPC_DELEGATE_STATE_PLAYING:
-    case AUDIO_OUTPUT_IPC_DELEGATE_STATE_PAUSED:
+    case mojom::AudioOutputStreamState::PLAYING:

[Henrik Grunell, 2016/09/01 15:09:07]

This seems to be an old IPC legacy. Play() and Pause() (and SetVolume()?) should
send a response instead. I don't think there are any cases where we could not
send a response that replaces usage of the "state changed" function.

+    case mojom::AudioOutputStreamState::PAUSED:
       break;
-    case AUDIO_OUTPUT_IPC_DELEGATE_STATE_ERROR:
+    case mojom::AudioOutputStreamState::ERROR:
       DLOG(WARNING) << "AudioOutputDevice::OnStateChanged(ERROR)";
       // Don't dereference the callback object if the audio thread
       // is stopped or stopping.  That could mean that the callback