Title: [193999] trunk/Source/WebCore
Revision
193999
Author
[email protected]
Date
2015-12-11 17:46:55 -0800 (Fri, 11 Dec 2015)

Log Message

[EME] Do not pass in the initialization data to AVContentKeyRequest as the contentIdentifier.
https://bugs.webkit.org/show_bug.cgi?id=152204
rdar://problem/23867877

Reviewed by Eric Carlson.

The AVContentKeyRequest API has been updated to no longer require a contentId parameter if the
ID can be derived from the initialization data.

* platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm:
(WebCore::CDMSessionAVContentKeySession::update):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (193998 => 193999)


--- trunk/Source/WebCore/ChangeLog	2015-12-12 01:32:43 UTC (rev 193998)
+++ trunk/Source/WebCore/ChangeLog	2015-12-12 01:46:55 UTC (rev 193999)
@@ -1,3 +1,17 @@
+2015-12-11  Jer Noble  <[email protected]>
+
+        [EME] Do not pass in the initialization data to AVContentKeyRequest as the contentIdentifier.
+        https://bugs.webkit.org/show_bug.cgi?id=152204
+        rdar://problem/23867877
+
+        Reviewed by Eric Carlson.
+
+        The AVContentKeyRequest API has been updated to no longer require a contentId parameter if the
+        ID can be derived from the initialization data.
+
+        * platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm:
+        (WebCore::CDMSessionAVContentKeySession::update):
+
 2015-12-11  Alexey Proskuryakov  <[email protected]>
 
         Roll out http://trac.webkit.org/r193984, because the new test is timing out.

Modified: trunk/Source/WebCore/platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm (193998 => 193999)


--- trunk/Source/WebCore/platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm	2015-12-12 01:32:43 UTC (rev 193998)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm	2015-12-12 01:46:55 UTC (rev 193999)
@@ -267,8 +267,6 @@
         ASSERT(m_keyRequest);
         RetainPtr<NSData> certificateData = adoptNS([[NSData alloc] initWithBytes:m_certificate->data() length:m_certificate->length()]);
 
-        RetainPtr<NSData> initData = adoptNS([[NSData alloc] initWithBytes:m_initData->data() length:m_initData->length()]);
-
         RetainPtr<NSDictionary> options;
         if (!m_protocolVersions.isEmpty() && canLoadAVContentKeyRequestProtocolVersionsKey()) {
             RetainPtr<NSMutableArray> protocolVersionsOption = adoptNS([[NSMutableArray alloc] init]);
@@ -283,7 +281,7 @@
 
         errorCode = MediaPlayer::NoError;
         systemCode = 0;
-        NSData* requestData = [m_keyRequest contentKeyRequestDataForApp:certificateData.get() contentIdentifier:initData.get() options:options.get() error:nil];
+        NSData* requestData = [m_keyRequest contentKeyRequestDataForApp:certificateData.get() contentIdentifier:nil options:options.get() error:nil];
         nextMessage = Uint8Array::create(static_cast<const uint8_t*>([requestData bytes]), [requestData length]);
 
         return false;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to