Improving WebM video duration estimation.

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 ee1abaf733334bd537e4bbb6b13190ca0b9d37c6..c41d5fe465d3e8ca8af4d81c0a8c0985b4c98b5b 100644
--- a/media/filters/source_buffer_stream.cc
+++ b/media/filters/source_buffer_stream.cc
@@ -786,8 +786,9 @@ 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.
-  if (splice_frames_enabled_)
+  if (splice_frames_enabled_) {

[wolenetz, 2015/03/28 00:26:06]

nit: why { } here and not elsewhere like below, and why in this CL?

[chcunningham, 2015/04/13 23:25:18]

Reverted. I see this file has plenty of single liners, so its good to be
consistent. I hate them, but I appear to be in the minority. 

[wolenetz, 2015/04/15 02:55:23]

No, I personally hope you're not in the minority on this.

     GenerateSpliceFrame(new_buffers);
+  }
 
   DecodeTimestamp prev_timestamp = last_appended_buffer_timestamp_;
   bool prev_is_keyframe = last_appended_buffer_is_keyframe_;