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

webkit/glue/simple_webmimeregistry_impl.cc

Index: webkit/glue/simple_webmimeregistry_impl.cc
diff --git a/webkit/glue/simple_webmimeregistry_impl.cc b/webkit/glue/simple_webmimeregistry_impl.cc
index b8e81a8b5c6385f2c8ab4cc308dea1e66beb9df5..5de1d54cf120c3f54de40bbb130039c4bc43115f 100644
--- a/webkit/glue/simple_webmimeregistry_impl.cc
+++ b/webkit/glue/simple_webmimeregistry_impl.cc
@@ -11,6 +11,7 @@
 #include "third_party/WebKit/Source/Platform/chromium/public/WebString.h"
 #include "webkit/base/file_path_string_conversions.h"
 #include "webkit/media/crypto/key_systems.h"
+#include "webkit/media/webmediasourceclient_impl.h"
 
 using WebKit::WebString;
 using WebKit::WebMimeRegistry;
@@ -106,6 +107,18 @@ WebMimeRegistry::SupportsType SimpleWebMimeRegistryImpl::supportsMediaMIMEType(
   return IsSupported;
 }
 
+bool SimpleWebMimeRegistryImpl::supportsMediaSourceMIMEType(
+    const WebKit::WebString& mime_type,
+    const WebString& codecs) {
+  const std::string mime_type_ascii = ToASCIIOrEmpty(mime_type);
+  std::vector<std::string> parsed_codec_ids;
+  net::ParseCodecString(ToASCIIOrEmpty(codecs), &parsed_codec_ids, false);
+  if (mime_type_ascii.empty() || parsed_codec_ids.size() == 0)
+    return false;
+  return webkit_media::WebMediaSourceClientImpl::isTypeSupported(

[scherkus (not reviewing), 2013/03/14 18:49:40]

why not call media::StreamParserFactory::IsTypeSupported(type, codecs) directly
instead of going through WMSCImpl?

[acolwell GONE FROM CHROMIUM, 2013/03/14 20:08:49]

git cl upload said "no!" I think it is because code in "webkit/glue" is not
allowed to depend on stuff in "media/". It explicitly complained about this.

[scherkus (not reviewing), 2013/03/14 20:21:03]

well that's because media/ isn't in webkit/glue/DEPS but you're working around
it via webkit/media/DEPS

I don't think the indirection is doing us much -- I think poking a small hole in
webkit/glue/DEPS to depend on media/filters/stream_parser_factory.h should be
fine (darin: please shout if you disagree!)

[acolwell GONE FROM CHROMIUM, 2013/03/14 20:56:35]

Done.

+      mime_type_ascii, parsed_codec_ids);
+}
+
 WebMimeRegistry::SupportsType
     SimpleWebMimeRegistryImpl::supportsNonImageMIMEType(
     const WebString& mime_type) {