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

media/filters/decoder_stream_traits.cc

 // Copyright 2014 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.
 
 #include "media/filters/decoder_stream_traits.h"
 
 #include "base/logging.h"
+#include "media/base/audio_buffer.h"
 #include "media/base/audio_decoder.h"
 #include "media/base/audio_decoder_config.h"
 #include "media/base/video_decoder.h"
 #include "media/base/video_decoder_config.h"
+#include "media/base/video_frame.h"
 
 namespace media {
 
 std::string DecoderStreamTraits<DemuxerStream::AUDIO>::ToString() {
   return "Audio";
 }
 
 void DecoderStreamTraits<DemuxerStream::AUDIO>::Initialize(
     DecoderType* decoder,
     const DecoderConfigType& config,
@@ skipping 23 matching lines @@
   statistics.audio_bytes_decoded = bytes_decoded;
   statistics_cb.Run(statistics);
 }
 
 DecoderStreamTraits<DemuxerStream::AUDIO>::DecoderConfigType
     DecoderStreamTraits<DemuxerStream::AUDIO>::GetDecoderConfig(
         DemuxerStream& stream) {
   return stream.audio_decoder_config();
 }
 
+scoped_refptr<DecoderStreamTraits<DemuxerStream::AUDIO>::OutputType>
+    DecoderStreamTraits<DemuxerStream::AUDIO>::CreateEOSOutput() {
+  return OutputType::CreateEOSBuffer();
+}
+
 std::string DecoderStreamTraits<DemuxerStream::VIDEO>::ToString() {
   return "Video";
 }
 
 void DecoderStreamTraits<DemuxerStream::VIDEO>::Initialize(
     DecoderType* decoder,
     const DecoderConfigType& config,
     bool low_delay,
     const PipelineStatusCB& status_cb,
     const OutputCB& output_cb) {
@@ skipping 22 matching lines @@
   statistics.video_bytes_decoded = bytes_decoded;
   statistics_cb.Run(statistics);
 }
 
 DecoderStreamTraits<DemuxerStream::VIDEO>::DecoderConfigType
     DecoderStreamTraits<DemuxerStream::VIDEO>::GetDecoderConfig(
         DemuxerStream& stream) {
   return stream.video_decoder_config();
 }
 
+scoped_refptr<DecoderStreamTraits<DemuxerStream::VIDEO>::OutputType>
+    DecoderStreamTraits<DemuxerStream::VIDEO>::CreateEOSOutput() {
+  return OutputType::CreateEOSFrame();

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

Do you really need CreateEOSOutput() in DecoderStreamTraits? I think you should
be able to call OutputType::CreateEOSFrame() directly in DecoderStream.

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

For audio, it's called CreateEOSBuffer(). CreateEOSOutput() basically wraps the
naming difference b/w audio and video.

+}
+
 }  // namespace media