Add support for real audio output to mojo.

mojo/services/html_viewer/webmediaplayer_factory.cc

 // Copyright 2014 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 "mojo/services/html_viewer/webmediaplayer_factory.h"
 
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "base/threading/thread.h"
 #include "media/audio/audio_manager.h"
 #include "media/audio/audio_manager_base.h"
-#include "media/audio/null_audio_sink.h"
+#include "media/audio/audio_output_stream_sink.h"
 #include "media/base/audio_hardware_config.h"
 #include "media/base/media.h"
 #include "media/base/media_log.h"
 #include "media/blink/null_encrypted_media_player_support.h"
 #include "media/blink/webmediaplayer_impl.h"
 #include "media/blink/webmediaplayer_params.h"
 #include "media/filters/gpu_video_accelerator_factories.h"
 
 namespace mojo {
 
@@ skipping 41 matching lines @@
 #endif
 }
 
 const media::AudioHardwareConfig&
 WebMediaPlayerFactory::GetAudioHardwareConfig() {
   return audio_hardware_config_;
 }
 
 scoped_refptr<media::AudioRendererSink>
 WebMediaPlayerFactory::CreateAudioRendererSink() {
-  // TODO(acolwell): Replace this with an AudioRendererSink implementation
-  // that actually talks to the audio device or an audio mojo service.
-  return new media::NullAudioSink(GetMediaThreadTaskRunner());
+  // TODO(dalecurtis): Replace this with an interface to an actual mojo service;

[jamesr, 2014/10/21 00:15:45]

can you tie this to a crbug that describes the issue and where people can track
progress?

[DaleCurtis, 2014/10/21 00:23:32]

Done.

+  // the AudioOutputStreamSink will not work in sandboxed processes.
+  return new media::AudioOutputStreamSink();
 }
 
 scoped_refptr<base::SingleThreadTaskRunner>
 WebMediaPlayerFactory::GetMediaThreadTaskRunner() {
   if (!media_thread_.IsRunning())
     media_thread_.Start();
 
   return media_thread_.message_loop_proxy();
 }
 
 }  // namespace mojo