Adopt AVContentKeyReportGroup
Created attachment 384657 [details] Patch
Created attachment 384716 [details] Patch
Comment on attachment 384716 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=384716&action=review > Source/WebCore/ChangeLog:11 > + AVContentKeySession to continue working for platform versions without report group support. "... existing usage of AVContentKeySession to continue working ..." isn't quite right. Maybe "... existing usage of AVContentKeySession continues to work ..."? > Source/WebCore/WebCore.xcodeproj/project.pbxproj:22955 > + CD53555723748E5900A613BD /* MediaToolboxSoftLink.cpp */, > + CD53555623748E5900A613BD /* MediaToolboxSoftLink.h */, Are these necessary?
(In reply to Eric Carlson from comment #3) > Comment on attachment 384716 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=384716&action=review > > > Source/WebCore/ChangeLog:11 > > + AVContentKeySession to continue working for platform versions without report group support. > > "... existing usage of AVContentKeySession to continue working ..." isn't > quite right. > > Maybe "... existing usage of AVContentKeySession continues to work ..."? Sure. > > Source/WebCore/WebCore.xcodeproj/project.pbxproj:22955 > > + CD53555723748E5900A613BD /* MediaToolboxSoftLink.cpp */, > > + CD53555623748E5900A613BD /* MediaToolboxSoftLink.h */, > > Are these necessary? Only if you want to be able to open MediaToolboxSoftLink.{cpp,h} in Xcode. :)
Created attachment 384750 [details] Patch for landing
Comment on attachment 384750 [details] Patch for landing Clearing flags on attachment: 384750 Committed r253064: <https://trac.webkit.org/changeset/253064>
All reviewed patches have been landed. Closing bug.
<rdar://problem/57605017>