Fix possible buffer emission during an abort.

content/renderer/media/websourcebuffer_impl.cc

Index: content/renderer/media/websourcebuffer_impl.cc
diff --git a/content/renderer/media/websourcebuffer_impl.cc b/content/renderer/media/websourcebuffer_impl.cc
index ff3ac5b0079eaad191862c5bce60da8687bba712..05ff9f5623bced6b172e833d8f1f6bf23a77cfa1 100644
--- a/content/renderer/media/websourcebuffer_impl.cc
+++ b/content/renderer/media/websourcebuffer_impl.cc
@@ -88,7 +88,9 @@ void WebSourceBufferImpl::append(
 }
 
 void WebSourceBufferImpl::abort() {

[wolenetz, 2014/05/06 00:04:32]

Upon further inspection of the MSE spec, and given the abort() for at least
mp2ts streams could cause further buffers to be processed by the coded frame
processing algorithm, it looks like the Blink interface for WSB::abort() needs
to pass a double* timestamp_offset, and we need to (conditionally) update it
only if we get a non-NULL ptr from Blink AND the abort() changed the TimeDelta
version of it. See ::append(), above, for a clear example of what I mean. Since
coded frame processing can change the timestamp offset, and since everything
below WSBI uses TimeDeltas (lower precision than Blink's double), we need to do
this. I've filed crbug.com/370229 accordingly. Please add a TODO(wolenetz) for
this here, referencing that bug.

[damienv1, 2014/05/06 00:29:00]

Added a comment which references this bug.

-  demuxer_->Abort(id_);
+  demuxer_->Abort(id_,
+                  append_window_start_, append_window_end_,
+                  &timestamp_offset_);
 }
 
 void WebSourceBufferImpl::remove(double start, double end) {