Bug 1363605 - Widevine should support SW_SECURE_CRYPTO video robustness. r?gerald draft
authorChris Pearce <cpearce@mozilla.com>
Wed, 10 May 2017 16:00:21 +1200
changeset 575202 e2c3e3dbbcaf2b57ddf9e32a2ceca771a56bf5c5
parent 571618 a748acbebbde373a88868dc02910fb2bc5e6a023
child 627859 074a122665cef3972dfc9089763e1efd6f621407
push id57994
push userbmo:cpearce@mozilla.com
push dateWed, 10 May 2017 04:08:54 +0000
reviewersgerald
bugs1363605
milestone55.0a1
Bug 1363605 - Widevine should support SW_SECURE_CRYPTO video robustness. r?gerald Turns out that Chrome treats the robustness values as SW_SECURE_DECODE to mean that SW_SECURE_CRYPTO is also supported. So we'd better follow suit... MozReview-Commit-ID: 6J68IsSQhyL
dom/media/eme/MediaKeySystemAccess.cpp
--- a/dom/media/eme/MediaKeySystemAccess.cpp
+++ b/dom/media/eme/MediaKeySystemAccess.cpp
@@ -305,16 +305,17 @@ GetSupportedKeySystems()
       widevine.mInitDataTypes.AppendElement(NS_LITERAL_STRING("webm"));
       widevine.mPersistentState = KeySystemFeatureSupport::Requestable;
       widevine.mDistinctiveIdentifier = KeySystemFeatureSupport::Prohibited;
       widevine.mSessionTypes.AppendElement(MediaKeySessionType::Temporary);
 #ifdef MOZ_WIDGET_ANDROID
       widevine.mSessionTypes.AppendElement(MediaKeySessionType::Persistent_license);
 #endif
       widevine.mAudioRobustness.AppendElement(NS_LITERAL_STRING("SW_SECURE_CRYPTO"));
+      widevine.mVideoRobustness.AppendElement(NS_LITERAL_STRING("SW_SECURE_CRYPTO"));
       widevine.mVideoRobustness.AppendElement(NS_LITERAL_STRING("SW_SECURE_DECODE"));
 #if defined(XP_WIN)
       // Widevine CDM doesn't include an AAC decoder. So if WMF can't
       // decode AAC, and a codec wasn't specified, be conservative
       // and reject the MediaKeys request, since we assume Widevine
       // will be used with AAC.
       if (WMFDecoderModule::HasAAC()) {
         widevine.mMP4.SetCanDecrypt(EME_CODEC_AAC);