Enable round-tripping and updating of WebSourceBufferImpl timestamp offset

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 a7d8d8a537803b4f9a2a8bdf42777af21720a5a0..1791867944d58d85ccd5af0c4b1d3ba313d71a58 100644
--- a/content/renderer/media/websourcebuffer_impl.cc
+++ b/content/renderer/media/websourcebuffer_impl.cc
@@ -5,6 +5,7 @@
 #include "content/renderer/media/websourcebuffer_impl.h"
 
 #include "base/float_util.h"
+#include "media/base/buffers.h"
 #include "media/filters/chunk_demuxer.h"
 
 namespace content {
@@ -60,7 +61,22 @@ blink::WebTimeRanges WebSourceBufferImpl::buffered() {
 }
 
 void WebSourceBufferImpl::append(const unsigned char* data, unsigned length) {
-  demuxer_->AppendData(id_, data, length);
+  bool offset_updated = append(data, length, NULL);
+  DCHECK(!offset_updated);
+}
+
+bool WebSourceBufferImpl::append(
+    const unsigned char* data,
+    unsigned length,
+    double* new_timestamp_offset) {
+  base::TimeDelta new_offset_if_updated;
+  demuxer_->AppendData(id_, data, length, &new_offset_if_updated);

[acolwell GONE FROM CHROMIUM, 2014/02/27 18:27:21]

Sorry to keep causing thrashing on this. Based on my comments on the Blink-side
changes, I think it makes sense to just pass the timestamp_offset into this
call. Since we wouldn't need to return a bool is should be pretty simple to just
have the real append code update the *timestamp_offset.

[wolenetz, 2014/02/27 21:48:49]

Done.

+  if (new_timestamp_offset && new_offset_if_updated != media::kNoTimestamp()) {
+    *new_timestamp_offset = new_offset_if_updated.InSecondsF();
+    return true;
+  }
+
+  return false;
 }
 
 void WebSourceBufferImpl::abort() {