Simplify AudioRendererImpl by using AudioDevice.

media/filters/audio_renderer_base.cc

Index: media/filters/audio_renderer_base.cc
===================================================================
--- media/filters/audio_renderer_base.cc	(revision 110348)
+++ media/filters/audio_renderer_base.cc	(working copy)
@@ -147,8 +147,11 @@
 void AudioRendererBase::ConsumeAudioSamples(scoped_refptr<Buffer> buffer_in) {
   base::AutoLock auto_lock(lock_);
   DCHECK(state_ == kPaused || state_ == kSeeking || state_ == kPlaying ||
-         state_ == kUnderflow || state_ == kRebuffering);
-  DCHECK_GT(pending_reads_, 0u);
+         state_ == kUnderflow || state_ == kRebuffering ||
+         state_ == kStopped);
+  if (!pending_reads_)

[acolwell GONE FROM CHROMIUM, 2011/11/17 19:16:12]

nit: Consider just moving the DCHECK_GT() and --pending_reads_ below the (state_
== kStopped) check below. I think that will produce the desired result and still
allow us to detect cases where we got unexpected calls while not stopped.

[vrk (LEFT CHROMIUM), 2011/12/02 22:54:46]

Since this is a nit and I'm not that familiar with the code, I will leave the CL
as-is if you don't mind!

(Also, scherkus has a CL out that will soon change this anyway.
http://codereview.chromium.org/8763010/)

+    return;
+
   --pending_reads_;
 
   // TODO(scherkus): this happens due to a race, primarily because Stop() is a