Media Capabilities: stricter checks for the media configuration inputs.

third_party/WebKit/Source/modules/media_capabilities/MediaCapabilities.cpp

 // Copyright 2017 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 "modules/media_capabilities/MediaCapabilities.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
 #include "modules/media_capabilities/MediaCapabilitiesInfo.h"
 #include "modules/media_capabilities/MediaConfiguration.h"
 #include "modules/media_capabilities/MediaDecodingConfiguration.h"
 #include "modules/media_capabilities/MediaEncodingConfiguration.h"
 #include "platform/bindings/ScriptState.h"
 #include "platform/network/ParsedContentType.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebMediaRecorderHandler.h"
 #include "public/platform/modules/media_capabilities/WebMediaCapabilitiesClient.h"
 #include "public/platform/modules/media_capabilities/WebMediaCapabilitiesInfo.h"
 #include "public/platform/modules/media_capabilities/WebMediaConfiguration.h"
 
 namespace blink {
 
 namespace {
 
+constexpr const char* kAudioMimeTypePrefix = "audio/";
+constexpr const char* kVideoMimeTypePrefix = "video/";

[chcunningham, 2017/05/10 21:10:13]

we may need to add in the HLS prefix "application/". See examples in mime code
here:

https://cs.chromium.org/chromium/src/media/base/mime_util_internal.cc?rcl=5fa...

IIUC, its not clear from the "application/" mime whether it will describe audio
or video, so it may just be an allowed option in both configs. 

[mlamouri (slow - plz ping), 2017/05/11 10:41:23]

That means we should update the specification, right? Will file a spec bug and
update the CL.

+constexpr const char* kCodecsMimeTypeParam = "codecs";
+
+bool IsValidMimeType(const String& content_type, const String& prefix) {
+  ParsedContentType parsed_content_type(content_type,
+                                        ParsedContentType::Mode::kStrict);
+  if (!parsed_content_type.IsValid())
+    return false;
+
+  if (!parsed_content_type.MimeType().StartsWith(prefix))
+    return false;
+
+  if (parsed_content_type.ParameterCount() > 1)
+    return false;
+
+  if (parsed_content_type.ParameterCount() == 1 &&
+      parsed_content_type.ParameterValueForName(kCodecsMimeTypeParam)
+          .IsNull()) {
+    return false;
+  }
+
+  return true;
+}
+
+bool IsValidMediaConfiguration(const MediaConfiguration& configuration) {
+  return configuration.hasAudio() || configuration.hasVideo();
+}
+
+bool IsValidVideoConfiguration(const VideoConfiguration& configuration) {
+  DCHECK(configuration.hasContentType());
+
+  if (!IsValidMimeType(configuration.contentType(), kVideoMimeTypePrefix))
+    return false;
+
+  DCHECK(configuration.hasFramerate());
+  if (!std::isfinite(configuration.framerate()) ||
+      configuration.framerate() <= 0) {
+    return false;
+  }
+
+  return true;
+}
+
+bool IsValidAudioConfiguration(const AudioConfiguration& configuration) {
+  DCHECK(configuration.hasContentType());
+
+  if (!IsValidMimeType(configuration.contentType(), kAudioMimeTypePrefix))
+    return false;
+
+  return true;
+}
+
 WebAudioConfiguration ToWebAudioConfiguration(
     const AudioConfiguration& configuration) {
   WebAudioConfiguration web_configuration;
 
   // |contentType| is mandatory.
   DCHECK(configuration.hasContentType());
   ParsedContentType parsed_content_type(configuration.contentType(),
                                         ParsedContentType::Mode::kStrict);
-
-  // TODO(chcunningham): Throw TypeError for invalid input.
-  // DCHECK(parsed_content_type.IsValid());
+  DCHECK(parsed_content_type.IsValid());
 
   DEFINE_STATIC_LOCAL(const String, codecs, ("codecs"));
   web_configuration.mime_type = parsed_content_type.MimeType().LowerASCII();
   web_configuration.codec = parsed_content_type.ParameterValueForName(codecs);
 
   // |channels| is optional and will be set to a null WebString if not present.
   web_configuration.channels = configuration.hasChannels()
                                    ? WebString(configuration.channels())
                                    : WebString();
 
   if (configuration.hasBitrate())
     web_configuration.bitrate = configuration.bitrate();
 
   if (configuration.hasSamplerate())
     web_configuration.samplerate = configuration.samplerate();
 
   return web_configuration;
 }
 
 WebVideoConfiguration ToWebVideoConfiguration(
     const VideoConfiguration& configuration) {
   WebVideoConfiguration web_configuration;
 
   // All the properties are mandatory.
   DCHECK(configuration.hasContentType());
   ParsedContentType parsed_content_type(configuration.contentType(),
                                         ParsedContentType::Mode::kStrict);
-
-  // TODO(chcunningham): Throw TypeError for invalid input.
-  // DCHECK(parsed_content_type.IsValid());
+  DCHECK(parsed_content_type.IsValid());
 
   DEFINE_STATIC_LOCAL(const String, codecs, ("codecs"));
   web_configuration.mime_type = parsed_content_type.MimeType().LowerASCII();
   web_configuration.codec = parsed_content_type.ParameterValueForName(codecs);
 
   DCHECK(configuration.hasWidth());
   web_configuration.width = configuration.width();
 
   DCHECK(configuration.hasHeight());
   web_configuration.height = configuration.height();
 
   DCHECK(configuration.hasBitrate());
   web_configuration.bitrate = configuration.bitrate();
 
   DCHECK(configuration.hasFramerate());
   web_configuration.framerate = configuration.framerate();
 
   return web_configuration;
 }
 
 WebMediaConfiguration ToWebMediaConfiguration(
-    const MediaConfiguration& configuration) {
+    const MediaDecodingConfiguration& configuration) {
   WebMediaConfiguration web_configuration;
 
+  // |type| is mandatory.
+  DCHECK(configuration.hasType());
+  if (configuration.type() == "file")
+    web_configuration.type = MediaConfigurationType::kFile;
+  else if (configuration.type() == "media-source")
+    web_configuration.type = MediaConfigurationType::kMediaSource;
+  else
+    NOTREACHED();
+
   if (configuration.hasAudio()) {
     web_configuration.audio_configuration =
         ToWebAudioConfiguration(configuration.audio());
+  }
+
+  if (configuration.hasVideo()) {
+    web_configuration.video_configuration =
+        ToWebVideoConfiguration(configuration.video());
+  }
+
+  return web_configuration;
+}
+
+WebMediaConfiguration ToWebMediaConfiguration(
+    const MediaEncodingConfiguration& configuration) {
+  WebMediaConfiguration web_configuration;
+
+  // TODO(mcasas): parse and set the type for encoding.
+
+  if (configuration.hasAudio()) {
+    web_configuration.audio_configuration =
+        ToWebAudioConfiguration(configuration.audio());
   }
 
   if (configuration.hasVideo()) {
     web_configuration.video_configuration =
         ToWebVideoConfiguration(configuration.video());
   }
 
   return web_configuration;
 }
 
 }  // anonymous namespace
 
 MediaCapabilities::MediaCapabilities() = default;
 
 ScriptPromise MediaCapabilities::decodingInfo(
     ScriptState* script_state,
     const MediaDecodingConfiguration& configuration) {
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::Create(script_state);
   ScriptPromise promise = resolver->Promise();
 
-  // |type| is mandatory.
-  DCHECK(configuration.hasType());
+  if (!IsValidMediaConfiguration(configuration)) {
+    resolver->Reject(V8ThrowException::CreateTypeError(
+        script_state->GetIsolate(),
+        "The configuration dictionary has neither |video| nor |audio| "
+        "specified and needs at least one of them."));
+    return promise;
+  }
+
+  if (configuration.hasVideo() &&
+      !IsValidVideoConfiguration(configuration.video())) {
+    resolver->Reject(V8ThrowException::CreateTypeError(
+        script_state->GetIsolate(),
+        "The video configuration dictionary is not valid."));
+    return promise;
+  }
+
+  if (configuration.hasAudio() &&
+      !IsValidAudioConfiguration(configuration.audio())) {
+    resolver->Reject(V8ThrowException::CreateTypeError(
+        script_state->GetIsolate(),
+        "The audio configuration dictionary is not valid."));
+    return promise;
+  }
 
   Platform::Current()->MediaCapabilitiesClient()->DecodingInfo(
       ToWebMediaConfiguration(configuration),
       WTF::MakeUnique<CallbackPromiseAdapter<MediaCapabilitiesInfo, void>>(
           resolver));
 
   return promise;
 }
 
 ScriptPromise MediaCapabilities::encodingInfo(
     ScriptState* script_state,
     const MediaEncodingConfiguration& configuration) {
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::Create(script_state);
   ScriptPromise promise = resolver->Promise();
 
-  if (!configuration.hasVideo() && !configuration.hasAudio()) {
-    resolver->Reject(DOMException::Create(
-        kSyntaxError,
+  if (!IsValidMediaConfiguration(configuration)) {
+    resolver->Reject(V8ThrowException::CreateTypeError(
+        script_state->GetIsolate(),
         "The configuration dictionary has neither |video| nor |audio| "
         "specified and needs at least one of them."));
     return promise;
   }
 
+  if (configuration.hasVideo() &&
+      !IsValidVideoConfiguration(configuration.video())) {
+    resolver->Reject(V8ThrowException::CreateTypeError(
+        script_state->GetIsolate(),
+        "The video configuration dictionary is not valid."));
+    return promise;
+  }
+
+  if (configuration.hasAudio() &&
+      !IsValidAudioConfiguration(configuration.audio())) {
+    resolver->Reject(V8ThrowException::CreateTypeError(
+        script_state->GetIsolate(),
+        "The audio configuration dictionary is not valid."));
+    return promise;
+  }
+
   std::unique_ptr<WebMediaRecorderHandler> handler =
       Platform::Current()->CreateMediaRecorderHandler();
   if (!handler) {
     resolver->Reject(DOMException::Create(
         kInvalidStateError,
         "Platform error: could not create MediaRecorderHandler."));
     return promise;
   }
 
   handler->EncodingInfo(
       ToWebMediaConfiguration(configuration),
       WTF::MakeUnique<CallbackPromiseAdapter<MediaCapabilitiesInfo, void>>(
           resolver));
   return promise;
 }
 
 DEFINE_TRACE(MediaCapabilities) {}
 
 }  // namespace blink