Add warning message for empty robustness.

third_party/WebKit/Source/modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.cpp

 // Copyright 2014 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 "config.h"
 #include "modules/encryptedmedia/NavigatorRequestMediaKeySystemAccess.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/frame/OriginsUsingFeatures.h"
 #include "core/frame/UseCounter.h"
+#include "core/inspector/ConsoleMessage.h"
 #include "modules/encryptedmedia/EncryptedMediaUtils.h"
 #include "modules/encryptedmedia/MediaKeySession.h"
 #include "modules/encryptedmedia/MediaKeySystemAccess.h"
 #include "modules/encryptedmedia/MediaKeysController.h"
 #include "platform/EncryptedMediaRequest.h"
 #include "platform/Logging.h"
 #include "platform/network/ParsedContentType.h"
+#include "public/platform/Platform.h"
 #include "public/platform/WebEncryptedMediaClient.h"
 #include "public/platform/WebEncryptedMediaRequest.h"
 #include "public/platform/WebMediaKeySystemConfiguration.h"
 #include "public/platform/WebMediaKeySystemMediaCapability.h"
 #include "public/platform/WebVector.h"
 #include "wtf/Vector.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
@@ skipping 64 matching lines @@
 
     ScriptPromise promise() { return m_resolver->promise(); }
 
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
         visitor->trace(m_resolver);
         EncryptedMediaRequest::trace(visitor);
     }
 
 private:
+    // For widevine key system, generate warning and report to UMA if
+    // |m_supportedConfigurations| contains any video capability with empty
+    // robustness string.
+    // TODO(xhwang): Remove after we handle empty robustness correctly.
+    // See http://crbug.com/482277
+    void checkVideoCapabilityRobustness() const;
+
     Member<ScriptPromiseResolver> m_resolver;
     const String m_keySystem;
     WebVector<WebMediaKeySystemConfiguration> m_supportedConfigurations;
 };
 
 MediaKeySystemAccessInitializer::MediaKeySystemAccessInitializer(ScriptState* scriptState, const String& keySystem, const HeapVector<MediaKeySystemConfiguration>& supportedConfigurations)
     : m_resolver(ScriptPromiseResolver::create(scriptState))
     , m_keySystem(keySystem)
     , m_supportedConfigurations(supportedConfigurations.size())
 {
@@ skipping 17 matching lines @@
         ASSERT(config.hasPersistentState());
         webConfig.persistentState = convertMediaKeysRequirement(config.persistentState());
         if (config.hasSessionTypes()) {
             webConfig.hasSessionTypes = true;
             webConfig.sessionTypes = convertSessionTypes(config.sessionTypes());
         }
         // If |label| is not present, it will be a null string.
         webConfig.label = config.label();
         m_supportedConfigurations[i] = webConfig;
     }
+
+    checkVideoCapabilityRobustness();
 }
 
 void MediaKeySystemAccessInitializer::requestSucceeded(WebContentDecryptionModuleAccess* access)
 {
     m_resolver->resolve(new MediaKeySystemAccess(m_keySystem, adoptPtr(access)));
     m_resolver.clear();
 }
 
 void MediaKeySystemAccessInitializer::requestNotSupported(const WebString& errorMessage)
 {
     m_resolver->reject(DOMException::create(NotSupportedError, errorMessage));
     m_resolver.clear();
 }
 
+void MediaKeySystemAccessInitializer::checkVideoCapabilityRobustness() const
+{
+    // Only check for widevine key system.
+    if (keySystem() != "com.widevine.alpha")
+        return;
+
+    bool hasVideoCapabilities = false;
+    bool hasEmptyRobustness = false;
+
+    for (const auto& config : m_supportedConfigurations) {
+        if (!config.hasVideoCapabilities)
+            continue;
+
+        hasVideoCapabilities = true;
+
+        for (const auto& capability : config.videoCapabilities) {
+            if (capability.robustness.isEmpty()) {
+                hasEmptyRobustness = true;
+                break;
+            }
+        }
+
+        if (hasEmptyRobustness)
+            break;
+    }
+
+    if (hasVideoCapabilities)
+        Platform::current()->histogramEnumeration("Media.EME.Widevine.VideoCapability.HasEmptyRobustness", hasEmptyRobustness, 2);
+
+    if (hasEmptyRobustness) {
+        m_resolver->executionContext()->addConsoleMessage(ConsoleMessage::create(JSMessageSource, WarningMessageLevel,
+            "It is recommended that a robustness level be specified. Not specifying the robustness level could "
+            "result in unexpected behavior in the future, potentially including failure to play."));
+    }
+}
+
 } // namespace
 
 ScriptPromise NavigatorRequestMediaKeySystemAccess::requestMediaKeySystemAccess(
     ScriptState* scriptState,
     Navigator& navigator,
     const String& keySystem,
     const HeapVector<MediaKeySystemConfiguration>& supportedConfigurations)
 {
     WTF_LOG(Media, "NavigatorRequestMediaKeySystemAccess::requestMediaKeySystemAccess()");
 
@@ skipping 18 matching lines @@
     ExecutionContext* executionContext = scriptState->executionContext();
     String errorMessage;
     if (executionContext->isSecureContext(errorMessage)) {
         UseCounter::count(executionContext, UseCounter::EncryptedMediaSecureOrigin);
     } else {
         UseCounter::countDeprecation(executionContext, UseCounter::EncryptedMediaInsecureOrigin);
         // TODO(ddorwin): Implement the following:
         // Reject promise with a new DOMException whose name is NotSupportedError.
     }
 
-
     // 5. Let origin be the origin of document.
     //    (Passed with the execution context in step 7.)
 
     // 6. Let promise be a new promise.
     Document* document = toDocument(executionContext);
     if (!document->page()) {
         return ScriptPromise::rejectWithDOMException(
             scriptState, DOMException::create(InvalidStateError, "The context provided is not associated with a page."));
     }
 
     MediaKeySystemAccessInitializer* initializer = new MediaKeySystemAccessInitializer(scriptState, keySystem, supportedConfigurations);
     ScriptPromise promise = initializer->promise();
 
     // 7. Asynchronously determine support, and if allowed, create and
     //    initialize the MediaKeySystemAccess object.
     MediaKeysController* controller = MediaKeysController::from(document->page());
     WebEncryptedMediaClient* mediaClient = controller->encryptedMediaClient(executionContext);
     mediaClient->requestMediaKeySystemAccess(WebEncryptedMediaRequest(initializer));
 
     // 8. Return promise.
     return promise;
 }
 
 } // namespace blink