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

media/formats/webm/webm_cluster_parser.h

 // 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.
 
 #ifndef MEDIA_FORMATS_WEBM_WEBM_CLUSTER_PARSER_H_
 #define MEDIA_FORMATS_WEBM_WEBM_CLUSTER_PARSER_H_
 
 #include <deque>
 #include <map>
 #include <set>
@@ skipping 31 matching lines @@
     Track(int track_num,
           bool is_video,
           base::TimeDelta default_duration,
           const LogCB& log_cb);
     ~Track();
 
     int track_num() const { return track_num_; }
 
     // If a buffer is currently held aside pending duration calculation, returns
     // its decode timestamp. Otherwise, returns kInfiniteDuration().
-    base::TimeDelta GetReadyUpperBound();
+    DecodeTimestamp GetReadyUpperBound();
 
     // Prepares |ready_buffers_| for retrieval. Prior to calling,
     // |ready_buffers_| must be empty. Moves all |buffers_| with timestamp

[wolenetz, 2014/08/08 21:30:05]

nit: s/timestamp/decode timestamp/

[acolwell GONE FROM CHROMIUM, 2014/08/11 17:05:06]

Done.

     // before |before_timestamp| to |ready_buffers_|, preserving their order.
-    void ExtractReadyBuffers(const base::TimeDelta before_timestamp);
+    void ExtractReadyBuffers(const DecodeTimestamp before_timestamp);
 
     const BufferQueue& ready_buffers() const { return ready_buffers_; }
 
     // If |last_added_buffer_missing_duration_| is set, updates its duration
     // relative to |buffer|'s timestamp, and adds it to |buffers_| and unsets
     // |last_added_buffer_missing_duration_|. Then, if |buffer| is missing
     // duration, saves |buffer| into |last_added_buffer_missing_duration_|, or
     // otherwise adds |buffer| to |buffers_|.
     bool AddBuffer(const scoped_refptr<StreamParserBuffer>& buffer);
 
@@ skipping 183 matching lines @@
   Track audio_;
   Track video_;
   TextTrackMap text_track_map_;
 
   // Subset of |text_track_map_| maintained by GetTextBuffers(), and cleared by
   // ClearTextTrackReadyBuffers(). Callers of GetTextBuffers() get a const-ref
   // to this member.
   TextBufferQueueMap text_buffers_map_;
 
   // Limits the range of buffers returned by Get{Audio,Video,Text}Buffers() to
   // this exclusive upper bound. Set to kNoTimestamp(), meaning not yet

[wolenetz, 2014/08/08 21:30:05]

nit: s/kNoTimestamp()/kNoDecodeTimestamp()/

[acolwell GONE FROM CHROMIUM, 2014/08/11 17:05:06]

Done.

   // calculated, by Reset() and Parse(). If kNoTimestamp(), then

[wolenetz, 2014/08/08 21:30:05]

nit: ditto

[acolwell GONE FROM CHROMIUM, 2014/08/11 17:05:06]

Done.

   // Get{Audio,Video,Text}Buffers() will calculate it to be the minimum (decode)
   // timestamp across all tracks' |last_buffer_missing_duration_|, or
   // kInfiniteDuration() if no buffers are currently missing duration.
-  base::TimeDelta ready_buffer_upper_bound_;
+  DecodeTimestamp ready_buffer_upper_bound_;
 
   LogCB log_cb_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(WebMClusterParser);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_FORMATS_WEBM_WEBM_CLUSTER_PARSER_H_