Audio and video decoders for MediaCodecPlayer

media/base/android/demuxer_stream_player_params.cc

 // Copyright (c) 2013 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/base/android/demuxer_stream_player_params.h"
 
 namespace media {
 
 DemuxerConfigs::DemuxerConfigs()
     : audio_codec(kUnknownAudioCodec),
       audio_channels(0),
       audio_sampling_rate(0),
       is_audio_encrypted(false),
       audio_codec_delay_ns(-1),
       audio_seek_preroll_ns(-1),
       video_codec(kUnknownVideoCodec),
       is_video_encrypted(false) {}
 
 DemuxerConfigs::~DemuxerConfigs() {}
 
 AccessUnit::AccessUnit() : is_end_of_stream(false), is_key_frame(false) {}
 
 AccessUnit::~AccessUnit() {}
 
 DemuxerData::DemuxerData() : type(DemuxerStream::UNKNOWN) {}
 
 DemuxerData::~DemuxerData() {}
 
 }  // namespace media
+
+std::ostream& operator<<(std::ostream& os, const media::AccessUnit& au) {
+  os << "status:" << au.status << (au.is_end_of_stream ? " EOS" : "")
+     << (au.is_key_frame ? " KEY_FRAME" : "") << " pts:" << au.timestamp

[wolenetz, 2015/06/19 22:46:42]

nit: it might also be interesting to output dts here as well, since pts isn't
guaranteed to be monotonically non-decreasing (like dts) in out-of-order decode
codecs (like h264).

[Tima Vaisburd, 2015/06/20 02:32:30]

Yes, I even have this situation in the unit test :-)

There is only one |timestamp| field in AccessUnit and I believe this is PTS.
Although there is no DTS, I think we know the decoding order, which is the order
the AU are coming from demuxer. At least this is the order I put them into
MediaCodec, so I made this assumption.
For the logs this order comes naturally as I print the chunk one AU after
another.

[wolenetz, 2015/06/22 21:41:21]

You're right. Ignore my comment - we don't need to add DTS to AU just for
logging, nor in this CL.

+     << " size:" << au.data.size();

[wolenetz, 2015/06/19 22:46:41]

nit: if an AU contains a config change, that would be especially interesting
data to log. Please add that here, too.

[Tima Vaisburd, 2015/06/20 02:32:30]

I added operator<< but used is so far in SetDemuxerConfigs only. Unfortunately
there is a duplication, could not use
https://code.google.com/p/chromium/codesearch#chromium/src/media/base/audio_d...
(and video) because they are not static.

[wolenetz, 2015/06/22 21:41:21]

Duplication is ugly; but unless we jump forward to a mojo-ized world where we
deserialize back into a {Audio,Video}DecoderConfig, then we'll use what you have
here.

+  return os;
+}