commit 50252de90c71b7d5d12c30121a82d4e790f74252
author mcasas <mcasas@chromium.org> Fri Jan 06 22:46:38 2017
committer Commit bot <commit-bot@chromium.org> Fri Jan 06 22:46:38 2017
tree 1822d9730ffc4ce79886ef71b66c57345af1a4e4
parent 444e0bbea061042b4133f3bfea0c8d0a6db73aff

WebmMuxer: do not set DefaultDuration if timestamp delta is not near constant

WebmMuxer sets WebM's DefaultDuration to the expected delta between
frames (derived from the frame rate), but that messes up (some) players
due to frames arriving at a different rate. This is a legimimate case
if, e.g.  the encoded data stream is paused.

This CL essentially avoids setting this parameter so that players
can just look at each encoded frame timestamp instead.

[1] http://www.webmproject.org/docs/container/#track

BUG=606000

Review-Url: https://codereview.chromium.org/2617143003
Cr-Commit-Position: refs/heads/master@{#442081}

media/muxers/webm_muxer.cc

diff --git a/media/muxers/webm_muxer.cc b/media/muxers/webm_muxer.cc
index 0dc0720..616e7b9 100644
--- a/media/muxers/webm_muxer.cc
+++ b/media/muxers/webm_muxer.cc
@@ -242,8 +242,6 @@
   DCHECK_EQ(0ull, video_track->crop_bottom());
   DCHECK_EQ(0.0f, video_track->frame_rate());
 
-  video_track->set_default_duration(base::Time::kNanosecondsPerSecond /
-                                    frame_rate);
   // Segment's timestamps should be in milliseconds, DCHECK it. See
   // http://www.webmproject.org/docs/container/#muxer-guidelines
   DCHECK_EQ(1000000ull, segment_.GetSegmentInfo()->timecode_scale());