Changeset 239373 in webkit

Timestamp:

Dec 19, 2018, 12:55:54 AM (8 years ago)

Author:

calvaris@igalia.com

Message:

[EME] MediaKeySystemConfiguration distinctiveIdentifier and persistentState should default to optional
​https://bugs.webkit.org/show_bug.cgi?id=192815

Reviewed by Jer Noble.

​https://www.w3.org/TR/encrypted-media/#dom-mediakeysystemconfiguration
says that distinctiveIdentifier and persistentState default to
optional. Our implementation does not define a default leaving it
to the first option of the enum, which currently is Required.

• platform/encryptedmedia/CDMKeySystemConfiguration.h:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/encryptedmedia/CDMKeySystemConfiguration.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-12-19  Xabier Rodriguez Calvar  <calvaris@igalia.com>
+
+        [EME] MediaKeySystemConfiguration distinctiveIdentifier and persistentState should default to optional
+        https://bugs.webkit.org/show_bug.cgi?id=192815
+
+        Reviewed by Jer Noble.
+
+        https://www.w3.org/TR/encrypted-media/#dom-mediakeysystemconfiguration
+        says that distinctiveIdentifier and persistentState default to
+        optional. Our implementation does not define a default leaving it
+        to the first option of the enum, which currently is Required.
+
+        * platform/encryptedmedia/CDMKeySystemConfiguration.h:
+
 2018-12-19  Rob Buis  <rbuis@igalia.com>
 

trunk/Source/WebCore/platform/encryptedmedia/CDMKeySystemConfiguration.h

@@ -46 +46 @@
     Vector<CDMMediaCapability> audioCapabilities;
     Vector<CDMMediaCapability> videoCapabilities;
-    CDMRequirement distinctiveIdentifier;
-    CDMRequirement persistentState;
+    CDMRequirement distinctiveIdentifier { CDMRequirement::Optional };
+    CDMRequirement persistentState { CDMRequirement::Optional };
     Vector<CDMSessionType> sessionTypes;
 };