Bug 185537 - REGRESSION(r231057): Encrypted media content playback failures
Summary: REGRESSION(r231057): Encrypted media content playback failures
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
Depends on: 184428
Blocks:
  Show dependency treegraph
 
Reported: 2018-05-10 17:26 PDT by Brent Fulgham
Modified: 2018-05-11 09:01 PDT (History)
2 users (show)

See Also:


Attachments
Patch (1.49 KB, patch)
2018-05-10 17:31 PDT, Brent Fulgham
no flags Details | Formatted Diff | Diff
Patch (1.51 KB, patch)
2018-05-10 21:39 PDT, Brent Fulgham
eric.carlson: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2018-05-10 17:26:30 PDT
Certain encrypted media content won't stream properly in WebKit if the WebContent process cannot validate the signature of its codec.

We removed a bunch of unneeded permissions in Bug 184428, but should hav let the SecurityServer connection available.
Comment 1 Brent Fulgham 2018-05-10 17:26:51 PDT
<rdar://problem/40038478>
Comment 2 Brent Fulgham 2018-05-10 17:31:28 PDT
Created attachment 340152 [details]
Patch
Comment 3 Jiewen Tan 2018-05-10 19:11:34 PDT
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?
Comment 4 Brent Fulgham 2018-05-10 19:34:16 PDT
(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.
Comment 5 Brent Fulgham 2018-05-10 21:39:27 PDT
Created attachment 340168 [details]
Patch
Comment 6 Brent Fulgham 2018-05-11 09:01:35 PDT
Committed r231702: <https://trac.webkit.org/changeset/231702>