Bug 210970 - [EME][CDMProxy] Default initialize m_numDecryptorsWaitingForKey member
Summary: [EME][CDMProxy] Default initialize m_numDecryptorsWaitingForKey member
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Charlie Turner
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-24 07:27 PDT by Charlie Turner
Modified: 2020-04-28 03:58 PDT (History)
9 users (show)

See Also:

Attachments
Patch (1.74 KB, patch)
2020-04-24 07:30 PDT, Charlie Turner 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Charlie Turner 2020-04-24 07:27:06 PDT 
[EME][CDMProxy] Default initialize m_numDecryptorsWaitingForKey member
Comment 1 Charlie Turner 2020-04-24 07:30:04 PDT 
Created attachment 397457 [details]
Patch
Comment 2 Xabier Rodríguez Calvar 2020-04-24 07:53:40 PDT 
Comment on attachment 397457 [details]
Patch

I think we should use wtf/Atomic instead.
Comment 3 Charlie Turner 2020-04-27 06:10:58 PDT 
(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.
Comment 4 EWS 2020-04-28 03:58:29 PDT 
Committed r260815: <https://trac.webkit.org/changeset/260815>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 397457 [details].