Bug 264859

Summary: Replace strongThis by protectedThis
Product: WebKit Reporter: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Component: WebCore Misc.Assignee: Jean-Yves Avenard [:jya] <jean-yves.avenard>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, Hironori.Fujii, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

Description Jean-Yves Avenard [:jya] 2023-11-14 20:19:37 PST 
From webkit coding style:
https://webkit.org/code-style-guidelines/#names-protectors-this

"Ref and RefPtr objects meant to protect this from deletion should be named "protectedThis".
Comment 1 Radar WebKit Bug Importer 2023-11-14 20:19:53 PST 
<rdar://problem/118434279>
Comment 2 Fujii Hironori 2023-11-14 20:27:13 PST 
Chris said we should update the style guideline in favor of the smart pointer guideline.

> we can get rid of protectedThis from our codebase

https://github.com/WebKit/WebKit/wiki/Smart-Pointer-Usage-Guidelines
Comment 3 Chris Dumez 2023-11-14 20:28:30 PST 
(In reply to Fujii Hironori from comment #2)
> Chris said we should update the style guideline in favor of the smart
> pointer guideline.
> 
> > we can get rid of protectedThis from our codebase
> 
> https://github.com/WebKit/WebKit/wiki/Smart-Pointer-Usage-Guidelines

Not in lambda captures, which I think is what Jean-Yves is talking about.
Comment 4 Jean-Yves Avenard [:jya] 2023-11-14 20:30:54 PST 
Pull request: https://github.com/WebKit/WebKit/pull/20520
Comment 5 EWS 2023-11-15 00:16:47 PST 
Committed 270757@main (c72b2982933d): <https://commits.webkit.org/270757@main>

Reviewed commits have been landed. Closing PR #20520 and removing active labels.