Summary: | [EME][CDMProxy] Default initialize m_numDecryptorsWaitingForKey member | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Charlie Turner <cturner> | ||||
Component: | WebKitGTK | Assignee: | Charlie Turner <cturner> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bugs-noreply, calvaris, dpino, eric.carlson, ews-watchlist, glenn, jer.noble, philipj, sergio | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=210390 | ||||||
Attachments: |
|
Description
Charlie Turner
2020-04-24 07:27:06 PDT
Created attachment 397457 [details]
Patch
Comment on attachment 397457 [details]
Patch
I think we should use wtf/Atomic instead.
(In reply to Xabier RodrÃguez Calvar from comment #2) > Comment on attachment 397457 [details] > Patch > > I think we should use wtf/Atomic instead. Why? That's only a wrapper around std::atomic providing conveniences for CAS that are not relevant to us. Committed r260815: <https://trac.webkit.org/changeset/260815> All reviewed patches have been landed. Closing bug and clearing flags on attachment 397457 [details]. |