Summary: | REGRESSION(r231057): Encrypted media content playback failures | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> | ||||||
Component: | WebKit2 | Assignee: | Brent Fulgham <bfulgham> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric.carlson, jiewen_tan | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 184428 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Brent Fulgham
2018-05-10 17:26:30 PDT
Created attachment 340152 [details]
Patch
Comment on attachment 340152 [details]
Patch
Thanks Brent for covering this regression. It looks like the block you are trying to add securityd to is restricted with:
!PLATFORM(MAC) || __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300
Should we add it to another block? Or to let it has its own block like?
(In reply to Jiewen Tan from comment #3) > Comment on attachment 340152 [details] > Patch > > Thanks Brent for covering this regression. It looks like the block you are > trying to add securityd to is restricted with: > !PLATFORM(MAC) || __MAC_OS_X_VERSION_MIN_REQUIRED >= 101300 > > Should we add it to another block? Or to let it has its own block like? Oh, that’s probably a good idea, since we removed it for all macOS releases. I’ll revise the patch. Created attachment 340168 [details]
Patch
Committed r231702: <https://trac.webkit.org/changeset/231702> |