Bug 226249

Summary: [GStreamer] Stop using UncheckedLock in WebKitCommonEncryptionDecryptorGStreamer
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKitGTKAssignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, bugs-noreply, calvaris, cgarcia, darin, ews-watchlist, ggaren, gustavo, mcatanzaro, menard, pnormand, sam, vjaquez
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 226212    
Bug Blocks:    
Attachments:
Description Flags
Patch
ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch darin: review+

Description Chris Dumez 2021-05-25 16:31:28 PDT
Stop using UncheckedLock in WebKitCommonEncryptionDecryptorGStreamer as it is being phased out in favor of WTF::Lock, which supports Clang thread safety analysis.
Comment 1 Chris Dumez 2021-05-25 16:33:32 PDT
Created attachment 429710 [details]
Patch
Comment 2 Chris Dumez 2021-05-25 17:02:09 PDT
Created attachment 429712 [details]
Patch
Comment 3 Chris Dumez 2021-05-25 17:38:34 PDT
Created attachment 429717 [details]
Patch
Comment 4 Darin Adler 2021-05-25 20:37:59 PDT
Comment on attachment 429717 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=429717&action=review

> Source/WebCore/platform/graphics/gstreamer/eme/WebKitCommonEncryptionDecryptorGStreamer.cpp:207
> +        // Note that waitFor() releases the lock lock internally while it waits, so isFlushing may have been changed.

"the lock lock"?
Comment 5 Chris Dumez 2021-05-25 21:58:34 PDT
Committed r278075 (238154@main): <https://commits.webkit.org/238154@main>