Introduce DecodeTimestamp class to make it easier to distiguish presentation and decode timestamps.

media/formats/mp2t/es_parser_adts.cc

Index: media/formats/mp2t/es_parser_adts.cc
diff --git a/media/formats/mp2t/es_parser_adts.cc b/media/formats/mp2t/es_parser_adts.cc
index 433baabe5e674017fcbb2b150fcaefb9006d0a37..08a54c437dd820f67c17bcdce57f4359af4432e9 100644
--- a/media/formats/mp2t/es_parser_adts.cc
+++ b/media/formats/mp2t/es_parser_adts.cc
@@ -127,7 +127,7 @@ EsParserAdts::~EsParserAdts() {
 
 bool EsParserAdts::Parse(const uint8* buf, int size,
                          base::TimeDelta pts,
-                         base::TimeDelta dts) {
+                         DecodeTimestamp dts) {
   // The incoming PTS applies to the access unit that comes just after
   // the beginning of |buf|.
   if (pts != kNoTimestamp())
@@ -166,7 +166,6 @@ bool EsParserAdts::Parse(const uint8* buf, int size,
             adts_frame.size,
             is_key_frame,
             DemuxerStream::AUDIO, 0);
-    stream_parser_buffer->SetDecodeTimestamp(current_pts);

[damienv1, 2014/08/14 23:11:33]

Should not be removed: most audio streams in Mpeg2 TS do not have dts, in that
case, dts is equal to the pts.

[wolenetz, 2014/08/14 23:32:51]

If not explicitly set (or if kNoDecodeTimestamp()), then dts is taken from pts.
So if you have no differing dts to set vs pts, you don't need to set it.

     stream_parser_buffer->set_timestamp(current_pts);
     stream_parser_buffer->set_duration(frame_duration);
     emit_buffer_cb_.Run(stream_parser_buffer);
@@ -266,4 +265,3 @@ bool EsParserAdts::UpdateAudioConfiguration(const uint8* adts_header) {
 
 }  // namespace mp2t
 }  // namespace media
-