diff options
| author | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-15 04:34:16 +0000 |
|---|---|---|
| committer | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-15 04:34:16 +0000 |
| commit | 8754d953bfca98c6dcc0d716dd8408afe5199248 (patch) | |
| tree | a397bfaf171848d4b4020d09318c2e49a7b0d2a8 /media/tools | |
| parent | b8ad39b6f840b7cb91a6d86f101ed1671a7e009f (diff) | |
| download | chromium_src-8754d953bfca98c6dcc0d716dd8408afe5199248.zip chromium_src-8754d953bfca98c6dcc0d716dd8408afe5199248.tar.gz chromium_src-8754d953bfca98c6dcc0d716dd8408afe5199248.tar.bz2 | |
Refactor FakeAudioOutputStream, NullAudioSink into FakeAudioConsumer.
We have two fake audio consumers: FakeAudioOutputStream on the browser
side and NullAudioSink on the renderer side. At present we can't remove
NullAudioSink since its used for PipelineIntegrationTests.
This change collapses the common functionality between both consumers
and as an added bonus replaces NullAudioSink's private thread in favor
of sharing the MediaThread.
BUG=none
TEST=unit tests, remote desktop session w/o audio, --disable-audio.
Review URL: https://chromiumcodereview.appspot.com/12334061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188251 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/tools')
| -rw-r--r-- | media/tools/player_x11/player_x11.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/tools/player_x11/player_x11.cc b/media/tools/player_x11/player_x11.cc index 6fc4e47..7b6e5c5 100644 --- a/media/tools/player_x11/player_x11.cc +++ b/media/tools/player_x11/player_x11.cc @@ -128,7 +128,7 @@ bool InitPipeline(const scoped_refptr<base::MessageLoopProxy>& message_loop, scoped_ptr<media::AudioRenderer> audio_renderer(new media::AudioRendererImpl( message_loop, - new media::NullAudioSink(), + new media::NullAudioSink(message_loop), media::SetDecryptorReadyCB())); collection->SetAudioRenderer(audio_renderer.Pass()); |