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

media/formats/mp4/mp4_stream_parser_unittest.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 <algorithm>
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/logging.h"
@@ skipping 14 matching lines @@
 namespace media {
 namespace mp4 {
 
 // TODO(xhwang): Figure out the init data type appropriately once it's spec'ed.
 static const char kMp4InitDataType[] = "video/mp4";
 
 class MP4StreamParserTest : public testing::Test {
  public:
   MP4StreamParserTest()
       : configs_received_(false),
-        lower_bound_(base::TimeDelta::Max()) {
+        lower_bound_(
+            DecodeTimestamp::FromPresentationTime(base::TimeDelta::Max())) {
     std::set<int> audio_object_types;
     audio_object_types.insert(kISO_14496_3);
     parser_.reset(new MP4StreamParser(audio_object_types, false));
   }
 
  protected:
   scoped_ptr<MP4StreamParser> parser_;
   bool configs_received_;
-  base::TimeDelta lower_bound_;
+  DecodeTimestamp lower_bound_;
 
   bool AppendData(const uint8* data, size_t length) {
     return parser_->Parse(data, length);
   }
 
   bool AppendDataInPieces(const uint8* data, size_t length, size_t piece_size) {
     const uint8* start = data;
     const uint8* end = data + length;
     while (start < end) {
       size_t append_size = std::min(piece_size,
@@ skipping 37 matching lines @@
     DumpBuffers("audio_buffers", audio_buffers);
     DumpBuffers("video_buffers", video_buffers);
 
     // TODO(wolenetz/acolwell): Add text track support to more MSE parsers. See
     // http://crbug.com/336926.
     if (!text_map.empty())
       return false;
 
     // Find the second highest timestamp so that we know what the
     // timestamps on the next set of buffers must be >= than.
-    base::TimeDelta audio = !audio_buffers.empty() ?
-        audio_buffers.back()->GetDecodeTimestamp() : kNoTimestamp();
-    base::TimeDelta video = !video_buffers.empty() ?
-        video_buffers.back()->GetDecodeTimestamp() : kNoTimestamp();
-    base::TimeDelta second_highest_timestamp =
-        (audio == kNoTimestamp() ||
-         (video != kNoTimestamp() && audio > video)) ? video : audio;
+    DecodeTimestamp audio = !audio_buffers.empty() ?
+        audio_buffers.back()->GetDecodeTimestamp() : kNoDecodeTimestamp();
+    DecodeTimestamp video = !video_buffers.empty() ?
+        video_buffers.back()->GetDecodeTimestamp() : kNoDecodeTimestamp();
+    DecodeTimestamp second_highest_timestamp =
+        (audio == kNoDecodeTimestamp() ||
+         (video != kNoDecodeTimestamp() && audio > video)) ? video : audio;
 
-    DCHECK(second_highest_timestamp != kNoTimestamp());
+    DCHECK(second_highest_timestamp != kNoDecodeTimestamp());
 
-    if (lower_bound_ != kNoTimestamp() &&
+    if (lower_bound_ != kNoDecodeTimestamp() &&
         second_highest_timestamp < lower_bound_) {
       return false;
     }
 
     lower_bound_ = second_highest_timestamp;
     return true;
   }
 
   void KeyNeededF(const std::string& type,
                   const std::vector<uint8>& init_data) {
     DVLOG(1) << "KeyNeededF: " << init_data.size();
     EXPECT_EQ(kMp4InitDataType, type);
     EXPECT_FALSE(init_data.empty());
   }
 
   void NewSegmentF() {
     DVLOG(1) << "NewSegmentF";
-    lower_bound_ = kNoTimestamp();
+    lower_bound_ = kNoDecodeTimestamp();
   }
 
   void EndOfSegmentF() {
     DVLOG(1) << "EndOfSegmentF()";
-    lower_bound_ = base::TimeDelta::Max();
+    lower_bound_ =
+        DecodeTimestamp::FromPresentationTime(base::TimeDelta::Max());
   }
 
   void InitializeParser() {
     parser_->Init(
         base::Bind(&MP4StreamParserTest::InitF, base::Unretained(this)),
         base::Bind(&MP4StreamParserTest::NewConfigF, base::Unretained(this)),
         base::Bind(&MP4StreamParserTest::NewBuffersF, base::Unretained(this)),
         true,
         base::Bind(&MP4StreamParserTest::KeyNeededF, base::Unretained(this)),
         base::Bind(&MP4StreamParserTest::NewSegmentF, base::Unretained(this)),
@@ skipping 95 matching lines @@
 // Delimiter (AUD) NALU.
 TEST_F(MP4StreamParserTest, VideoSamplesStartWithAUDs) {
   ParseMP4File("bear-1280x720-av_with-aud-nalus_frag.mp4", 512);
 }
 
 // TODO(strobe): Create and test media which uses CENC auxiliary info stored
 // inside a private box
 
 }  // namespace mp4
 }  // namespace media