Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(656)

Unified Diff: media/formats/webm/webm_cluster_parser.h

Issue 447963003: Introduce DecodeTimestamp class to make it easier to distiguish presentation and decode timestamps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/formats/webm/webm_cluster_parser.h
diff --git a/media/formats/webm/webm_cluster_parser.h b/media/formats/webm/webm_cluster_parser.h
index ab1d4a11fb11505ba34f664b840bb998c17521ab..caf42f4b9c3d96109e0ebefbc5268a10baf8cf40 100644
--- a/media/formats/webm/webm_cluster_parser.h
+++ b/media/formats/webm/webm_cluster_parser.h
@@ -49,12 +49,12 @@ class MEDIA_EXPORT WebMClusterParser : public WebMParserClient {
// 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_; }
@@ -263,7 +263,7 @@ class MEDIA_EXPORT WebMClusterParser : public WebMParserClient {
// 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_;

Powered by Google App Engine
This is Rietveld 408576698