Make VideoDecoder use (kAborted, NULL) to signify an aborted decode, instead of (kOk, NULL).

media/filters/video_frame_stream.cc

Index: media/filters/video_frame_stream.cc
diff --git a/media/filters/video_frame_stream.cc b/media/filters/video_frame_stream.cc
index 0c7ccf367992165c1f5cbf62ec028b3df752503d..77900974c1933b1e72d8b4a53fe6b489de9168f3 100644
--- a/media/filters/video_frame_stream.cc
+++ b/media/filters/video_frame_stream.cc
@@ -262,6 +262,12 @@ void VideoFrameStream::OnFrameReady(int buffer_size,
     return;
   }
 
+  if (status == VideoDecoder::kAborted) {
+    DCHECK(!frame.get());
+    AbortRead();
+    return;
+  }

[xhwang, 2014/01/03 01:05:58]

VideoDecoder::kAborted can only be a result of a previous VideoDecoder::Reset()
call, which is handled on l.280. This is why we didn't check for (kOk, NULL) in
the original impl. Sorry for not making this obvious :( Feel free to
refactor/clean-up/comment the code to make it cleaner.

Note that kAborted -> !reset_cb_.is_null() but not vice versa. For example, we
could have:
- Decode()
- Decode finished and OnFrameReady posted.
- Reset()
- OnFrameReady(kOk, valid_frame)  // !reset_cb_.is_null()
- Reset callback fired..

+
   // Any successful decode counts!
   if (buffer_size > 0) {
     PipelineStatistics statistics;