-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adopt CoreMedia API to use media parser daemon #5588
Adopt CoreMedia API to use media parser daemon #5588
Conversation
EWS run on previous version of this PR (hash 349448a) |
process' => process's (three s's in a row is weird! but correct.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me
Thanks for reviewing! |
349448a
to
dc1a2f1
Compare
Fixed! |
EWS run on current version of this PR (hash dc1a2f1) |
https://bugs.webkit.org/show_bug.cgi?id=246805 rdar://99653449 Reviewed by Geoffrey Garen. Adopt CoreMedia API to use media parser daemon. This requires access to the daemon in the GPU process's sandbox. * Source/WebCore/PAL/pal/spi/cocoa/AVStreamDataParserSPI.h: * Source/WebCore/platform/graphics/avfoundation/objc/SourceBufferParserAVFObjC.mm: (WebCore::SourceBufferParserAVFObjC::SourceBufferParserAVFObjC): * Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb.in: Canonical link: https://commits.webkit.org/255878@main
dc1a2f1
to
8170bc8
Compare
Committed 255878@main (8170bc8): https://commits.webkit.org/255878@main Reviewed commits have been landed. Closing PR #5588 and removing active labels. |
8170bc8
dc1a2f1
π§ͺ api-mac