MSE: Populate WebM missing duration with DefaultDuration, derived, or default

media/filters/source_buffer_stream.cc

Index: media/filters/source_buffer_stream.cc
diff --git a/media/filters/source_buffer_stream.cc b/media/filters/source_buffer_stream.cc
index a7d3ecf1f0a86eced8aa80623a3d46c81afc553c..df504aa6befae7ef05cf1695cfb71a6c0b5e1609 100644
--- a/media/filters/source_buffer_stream.cc
+++ b/media/filters/source_buffer_stream.cc
@@ -965,7 +965,13 @@ void SourceBufferStream::PrepareRangesForNextAppend(
   // Handle splices between the existing buffers and the new buffers.  If a
   // splice is generated the timestamp and duration of the first buffer in
   // |new_buffers| will be modified.
-  GenerateSpliceFrame(new_buffers);
+
+  // TODO(dalecurtis): GenerateSpliceFrame() causes crash in
+  // ChunkDemuxerTest.OutOfOrderClusters when all frames in a cluster have

[acolwell GONE FROM CHROMIUM, 2014/03/26 23:32:04]

nit: Remove comment since Dale has already disabled the generation for now and I
believe he's aware of the bug mentioned here.

[wolenetz, 2014/03/27 18:01:48]

Done (and rebased onto ToT that now includes Dale's
https://codereview.chromium.org/213283003/ that disables GenerateSpliceFrame()
outside of scoped tests.

+  // duration, including SimpleBlock frames whose duration is derived from
+  // a subsequent BlockGroup frame's timestamp for the same track in the
+  // cluster. See http://crbug.com/356805.
+  // GenerateSpliceFrame(new_buffers);
 
   base::TimeDelta prev_timestamp = last_appended_buffer_timestamp_;
   bool prev_is_keyframe = last_appended_buffer_is_keyframe_;