Add Chromium-side changes for MediaSource::isTypeSupported()

webkit/media/webmediasourceclient_impl.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/media/webmediasourceclient_impl.h"
 
 #include "base/guid.h"
 #include "media/filters/chunk_demuxer.h"
+#include "media/filters/stream_parser_factory.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebCString.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebString.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebSourceBuffer.h"
 
 using ::WebKit::WebString;
 using ::WebKit::WebMediaSourceClient;
 
 namespace webkit_media {
 
 #define COMPILE_ASSERT_MATCHING_STATUS_ENUM(webkit_name, chromium_name) \
@@ skipping 58 matching lines @@
   base::TimeDelta time_offset = base::TimeDelta::FromMicroseconds(
       offset * base::Time::kMicrosecondsPerSecond);
   return demuxer_->SetTimestampOffset(id_, time_offset);
 }
 
 void WebSourceBufferImpl::removedFromMediaSource() {
   demuxer_->RemoveId(id_);
   demuxer_ = NULL;
 }
 
+bool WebMediaSourceClientImpl::isTypeSupported(
+    const WebKit::WebString& type,
+    const WebKit::WebVector<WebKit::WebString>& codecs) {
+  std::vector<std::string> new_codecs(codecs.size());
+  for (size_t i = 0; i < codecs.size(); ++i)
+    new_codecs[i] = codecs[i].utf8().data();
+  return media::StreamParserFactory::IsTypeSupported(type.utf8().data(),
+                                                     new_codecs);
+}
+
 WebMediaSourceClientImpl::WebMediaSourceClientImpl(
-    const scoped_refptr<media::ChunkDemuxer>& demuxer)
-    : demuxer_(demuxer) {
+    const scoped_refptr<media::ChunkDemuxer>& demuxer,
+    media::LogCB log_cb)
+    : demuxer_(demuxer),
+      log_cb_(log_cb) {
   DCHECK(demuxer_);
 }
 
 WebMediaSourceClientImpl::~WebMediaSourceClientImpl() {}
 
 WebMediaSourceClient::AddStatus WebMediaSourceClientImpl::addSourceBuffer(
     const WebKit::WebString& type,
     const WebKit::WebVector<WebKit::WebString>& codecs,
     WebKit::WebSourceBuffer** source_buffer) {
+  DCHECK(isTypeSupported(type, codecs));
+
   std::string id = base::GenerateGUID();
+  bool has_audio = false;
+  bool has_video = false;
   std::vector<std::string> new_codecs(codecs.size());
   for (size_t i = 0; i < codecs.size(); ++i)
     new_codecs[i] = codecs[i].utf8().data();
+  scoped_ptr<media::StreamParser> stream_parser(
+      media::StreamParserFactory::Create(type.utf8().data(), new_codecs,
+                                         log_cb_, &has_audio, &has_video));
+
+  if (!stream_parser)
+    return AddStatusNotSupported;
 
   WebMediaSourceClient::AddStatus result =
       static_cast<WebMediaSourceClient::AddStatus>(
-          demuxer_->AddId(id, type.utf8().data(), new_codecs));
+          demuxer_->AddParser(id, stream_parser.Pass(), has_audio, has_video));

[scherkus (not reviewing), 2013/03/14 01:15:26]

where is ChunkDemuxer::AddParser() defined? I'm not seeing it anywhere :\

(I was curious if has_audio/has_video can be part of stream_parser instead of
being carried around along the side as out-params in SPF::Create() and
CD::AddParser())

[acolwell GONE FROM CHROMIUM, 2013/03/14 18:34:34]

This was accidentally left in for a refactor that I later decided to revert.
I've put this back to AddId() for now.

The has_audio/has_video information doesn't really belong in the StreamParser.
It is only used by the ChunkDemuxer to enforce one video and one audio stream. I
expect this to go away in a later refactoring CL where I extract the
stream_parser and SourceBufferStream code from ChunkDemuxer and move it here.

 
   if (result == WebMediaSourceClient::AddStatusOk)
     *source_buffer = new WebSourceBufferImpl(id, demuxer_);
 
   return result;
 }
 
 double WebMediaSourceClientImpl::duration() {
   double duration = demuxer_->GetDuration();
 
@@ skipping 33 matching lines @@
       break;
     default:
       NOTIMPLEMENTED();
   }
 
   if (!demuxer_->EndOfStream(pipeline_status))
     DVLOG(1) << "EndOfStream call failed.";
 }
 
 }  // namespace webkit_media