No EOS frame in {Audio|Video}Decoder::OutputCB.

media/base/video_decoder.h

 // Copyright (c) 2012 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.
 
 #ifndef MEDIA_BASE_VIDEO_DECODER_H_
 #define MEDIA_BASE_VIDEO_DECODER_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "media/base/media_export.h"
@@ skipping 11 matching lines @@
   // Status codes for decode operations on VideoDecoder.
   // TODO(rileya): Now that both AudioDecoder and VideoDecoder Status enums
   // match, break them into a decoder_status.h.
   enum Status {
     kOk,  // Everything went as planned.
     kAborted,  // Decode was aborted as a result of Reset() being called.
     kDecodeError,  // Decoding error happened.
     kDecryptError  // Decrypting error happened.
   };
 
-  // Callback to return decode frames.
+  // Callback for VideoDecoder to return a decoded frame whenever it becomes
+  // available. Only valid frames should be returned via this callback. EOS
+  // frames should not be returned via this callback.

[Sergey Ulanov, 2014/06/16 23:49:32]

same here as in audio_decoder.h

[xhwang, 2014/06/17 18:56:35]

Done.

   typedef base::Callback<void(const scoped_refptr<VideoFrame>&)> OutputCB;
 
   // Callback type for Decode(). Called after the decoder has completed decoding
   // corresponding DecoderBuffer, indicating that it's ready to accept another
   // buffer to decode.
   typedef base::Callback<void(Status status)> DecodeCB;
 
   VideoDecoder();
   virtual ~VideoDecoder();
 
@@ skipping 58 matching lines @@
   // Returns maximum number of parallel decode requests.
   virtual int GetMaxDecodeRequests() const;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(VideoDecoder);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_DECODER_H_