WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 137710
[Win] Enable Encrypted Media Support
https://bugs.webkit.org/show_bug.cgi?id=137710
Summary
[Win] Enable Encrypted Media Support
Brent Fulgham
Reported
2014-10-14 12:23:03 PDT
This patch makes the following changes: 1. Activate the ENCRYPTED_MEDIA_V2 support on Windows. 2. Handle possible calls to PlatformCALayerWinInternal::displayCallback on background queue. We do not support performing display operations off the main thread, but some callbacks are generated in background queues. We need to re-dispatch such calls on the main thread to avoid violating this fundamental rule for our layout design.
Attachments
Patch
(9.41 KB, patch)
2014-10-14 12:30 PDT
,
Brent Fulgham
jer.noble
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2014-10-14 12:23:23 PDT
<
rdar://problem/18652699
>
Brent Fulgham
Comment 2
2014-10-14 12:30:44 PDT
Created
attachment 239817
[details]
Patch
Jer Noble
Comment 3
2014-10-14 12:38:02 PDT
Comment on
attachment 239817
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=239817&action=review
r=me, with nit:
> Source/WebCore/platform/graphics/ca/win/PlatformCALayerWinInternal.cpp:72 > +struct displayOnMainThreadContext {
Nit: displayOnMainThreadContext -> DisplayOnMainThreadContext
Brent Fulgham
Comment 4
2014-10-14 12:51:39 PDT
Comment on
attachment 239817
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=239817&action=review
>> Source/WebCore/platform/graphics/ca/win/PlatformCALayerWinInternal.cpp:72 >> +struct displayOnMainThreadContext { > > Nit: displayOnMainThreadContext -> DisplayOnMainThreadContext
It shall be done!
Brent Fulgham
Comment 5
2014-10-14 13:11:29 PDT
Committed
r174698
: <
http://trac.webkit.org/changeset/174698
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug