Let WebSourceBufferImpl::setMode() update SourceState::sequence_mode_

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 c9429779d827de2145ffedaa10b58e13746704a6..eb54709fc82ea6e564a0984ba49634b8ad3354c4 100644
--- a/content/renderer/media/websourcebuffer_impl.cc
+++ b/content/renderer/media/websourcebuffer_impl.cc
@@ -36,6 +36,21 @@ WebSourceBufferImpl::~WebSourceBufferImpl() {
   DCHECK(!demuxer_) << "Object destroyed w/o removedFromMediaSource() call";
 }
 
+bool WebSourceBufferImpl::setMode(WebSourceBuffer::AppendMode mode) {
+  media::ChunkDemuxer::AppendMode demuxerMode = media::ChunkDemuxer::kSegments;
+  switch (mode) {
+    case WebSourceBuffer::AppendModeSegments:
+      break;
+    case WebSourceBuffer::AppendModeSequence:
+      demuxerMode = media::ChunkDemuxer::kSequence;
+      break;
+    default:

[acolwell GONE FROM CHROMIUM, 2014/01/10 23:10:37]

nit: Change switch to a cast and use COMPILE_ASSERTs to verify that the Blink
and ChunkDemuxer contants match. See WebMediaSourceImpl for an example.

[wolenetz, 2014/01/11 00:55:44]

Done using your other suggestion (use a bool |is_sequence_mode| since there are
only two valid modes). As such, there is no need for COMPILE_ASSERTs, and I
followed the example of using switch from WebMediaSourceImpl::markEndOfStream.

+      NOTIMPLEMENTED();
+  }
+
+  return demuxer_->SetAppendMode(id_, demuxerMode);
+}
+
 blink::WebTimeRanges WebSourceBufferImpl::buffered() {
   media::Ranges<base::TimeDelta> ranges = demuxer_->GetBufferedRanges(id_);
   blink::WebTimeRanges result(ranges.size());