Bug 237022

Summary: Thread safety analysis macros are confusing for non-Lock use-cases
Product: WebKit Reporter: Kimmo Kinnunen <kkinnunen>
Component: Web Template FrameworkAssignee: Kimmo Kinnunen <kkinnunen>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, cdumez, cmarcelo, dino, eric.carlson, ews-watchlist, glenn, jbedard, jean-yves.avenard, jer.noble, philipj, rniwa, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 236832    
Attachments:
Description Flags
Patch
none
Option B without the renames none

Description Kimmo Kinnunen 2022-02-22 00:09:38 PST
Thread safety analysis macros are confusing for non-Lock use-cases
Comment 1 Kimmo Kinnunen 2022-02-22 00:37:25 PST
Created attachment 452838 [details]
Patch
Comment 2 Kimmo Kinnunen 2022-02-22 04:07:22 PST
Created attachment 452855 [details]
Option B without the renames
Comment 3 Chris Dumez 2022-02-22 07:21:44 PST
Comment on attachment 452855 [details]
Option B without the renames

r=me
Comment 4 Dean Jackson 2022-02-22 09:03:09 PST
Comment on attachment 452838 [details]
Patch

Removing the r? flag from the first patch.
Comment 5 Jonathan Bedard 2022-02-22 16:44:29 PST
Stopped https://ews-build.webkit.org/#/builders/70/builds/1025, the 3 failures it flagged are already known.
Comment 6 Jonathan Bedard 2022-02-22 16:47:27 PST
The 3 failures flagged by https://ews-build.webkit.org/#/builders/70/builds/1025 are already known, so I stopped the build.
Comment 7 EWS 2022-02-23 02:00:25 PST
Committed r290361 (247679@main): <https://commits.webkit.org/247679@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 452855 [details].
Comment 8 Radar WebKit Bug Importer 2022-02-23 02:01:23 PST
<rdar://problem/89344972>