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

Unified Diff: media/filters/chunk_demuxer.cc

Issue 205703003: MSE: Use frame duration, if available, in LegacyFrameProcessor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased to pull in WebM frame duration estimation, undid the CD tests' conversion to just BlockGrou… Created 6 years, 9 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/filters/chunk_demuxer.cc
diff --git a/media/filters/chunk_demuxer.cc b/media/filters/chunk_demuxer.cc
index 1eb22d54f7f3334955189a97ac312db927d7d370..caf15ba9b2bc8e56fe80919bc651829b43b28490 100644
--- a/media/filters/chunk_demuxer.cc
+++ b/media/filters/chunk_demuxer.cc
@@ -1559,18 +1559,16 @@ void ChunkDemuxer::UpdateDuration(TimeDelta new_duration) {
host_->SetDuration(new_duration);
}
-void ChunkDemuxer::IncreaseDurationIfNecessary(
- TimeDelta last_appended_buffer_timestamp,
- ChunkDemuxerStream* stream) {
- DCHECK(last_appended_buffer_timestamp != kNoTimestamp());
- if (last_appended_buffer_timestamp <= duration_)
+void ChunkDemuxer::IncreaseDurationIfNecessary(TimeDelta new_duration) {
+ DCHECK(new_duration != kNoTimestamp());
+
+ if (new_duration <= duration_)
return;
- TimeDelta stream_duration = stream->GetBufferedDuration();
- DCHECK(stream_duration > TimeDelta());
+ DVLOG(2) << __FUNCTION__ << ": Increasing duration: "
+ << duration_.InSecondsF() << " -> " << new_duration.InSecondsF();
- if (stream_duration > duration_)
- UpdateDuration(stream_duration);
+ UpdateDuration(new_duration);
}
void ChunkDemuxer::DecreaseDurationIfNecessary() {

Powered by Google App Engine
This is Rietveld 408576698