Make the timestamp unroll offset consistent accross PES pids.

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 70b38fb7f61eda536eaa146619b44935c522e197..bb9433220cce02dd3838b419eb2674d88e3e8caf 100644
--- a/media/formats/mp2t/es_parser_adts.cc
+++ b/media/formats/mp2t/es_parser_adts.cc
@@ -159,6 +159,8 @@ bool EsParserAdts::ParseFromEsQueue() {
             is_key_frame,
             DemuxerStream::AUDIO, 0);
     stream_parser_buffer->set_timestamp(current_pts);
+    stream_parser_buffer->SetDecodeTimestamp(

[wolenetz, 2014/09/05 21:18:49]

If not set, buffer's DTS will be assumed to be PTS. See
StreamParserBuffer::GetDecodeTimestamp()
(https://code.google.com/p/chromium/codesearch#chromium/src/media/base/stream_...).
These two lines are unnecessary.

[damienv1, 2014/09/05 21:52:03]

Yes, I know. But this implicit DTS assignment is not correctly implemented in
DecodeTimestamp.
e.g.
Let's say I want to add an offset on both the pts and dts outside of this file,
I'll do:
set_timestamp(timestamp() + offset)
SetDecodeTimestamp(DecodeTimestamp() + offset)
and suddenly the DTS is applied twice the offset !

So there is sth wrong in the design of PTS/DTS inside StreamParserBuffer...

+        DecodeTimestamp::FromPresentationTime(current_pts));
     stream_parser_buffer->set_duration(frame_duration);
     emit_buffer_cb_.Run(stream_parser_buffer);