Report VideoDecoder status through ReadCB instead of through FilterHost.

media/filters/video_renderer_base.h

Index: media/filters/video_renderer_base.h
diff --git a/media/filters/video_renderer_base.h b/media/filters/video_renderer_base.h
index 2746ccb98da0e155383670477fbed0cbc380a0c2..af75de1ce065187b1ac94fb907bda73dba39c25a 100644
--- a/media/filters/video_renderer_base.h
+++ b/media/filters/video_renderer_base.h
@@ -74,8 +74,11 @@ class MEDIA_EXPORT VideoRendererBase
   void PutCurrentFrame(scoped_refptr<VideoFrame> frame);
 
  private:
-  // Callback from the video decoder delivering decoded video frames.
-  void FrameReady(scoped_refptr<VideoFrame> frame);
+  // Callback from the video decoder delivering decoded video frames and
+  // reporting video decoder status. If |status| is not VideoDecoder::kOk,
+  // it means some error has occurred in video decoder. In this case, |frame|
+  // should be NULL.

[scherkus (not reviewing), 2012/04/27 18:27:56]

nit: this documentation should be in the definition of VideoDecoder::ReadCB
isntead of being duplicated here

[xhwang, 2012/04/27 23:22:30]

Done.

+  void FrameReady(scoped_refptr<VideoFrame> frame, VideoDecoder::Status status);
 
   // Helper method that schedules an asynchronous read from the decoder as long
   // as there isn't a pending read and we have capacity.