Bug 226133

Summary:

Replace LockHolder with Locker in local variables

Product:

WebKit

Reporter:

Chris Dumez <cdumez>

Component:

WebCore Misc.

Assignee:

Chris Dumez <cdumez>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aakash_jain, aboxhall, achristensen, apinheiro, benjamin, berto, calvaris, cfleizach, cgarcia, cmarcelo, commit-queue, darin, ddkilzer, dmazzoni, eric.carlson, ews-watchlist, fred.wang, galpeter, ggaren, glenn, gustavo, gyuyoung.kim, hi, hta, jamesr, jcraig, jdiggs, jer.noble, joepeck, keith_miller, kkinnunen, kondapallykalyan, luiz, mark.lam, menard, msaboff, philipj, pnormand, ryuan.choi, saam, samuel_white, sam, sergio, tommyw, tonikitoo, tzagallo, vjaquez, webkit-bug-importer, zeno

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

URL:

https://en.cppreference.com/w/cpp/language/class_template_argument_deduction

See Also:

https://bugs.webkit.org/show_bug.cgi?id=226139

Bug Depends on:

226140

Bug Blocks:

226138

Attachments:

Description	Flags
Patch	ews-feeder: commit-queue-
Patch	ews-feeder: commit-queue-
Patch	ews-feeder: commit-queue-
Patch	ews-feeder: commit-queue-
Patch	none

Chris Dumez

Reported 2021-05-21 20:04:43 PDT

Replace LockHolder with Locker in local variables. It is shorter and it allows switching the lock type more easily since the compiler with deduce the lock type T for Locker<T>.

Attachments
Patch (276.14 KB, patch) 2021-05-21 20:12 PDT, Chris Dumez	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch (276.17 KB, patch) 2021-05-21 20:20 PDT, Chris Dumez	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch (276.78 KB, patch) 2021-05-21 21:16 PDT, Chris Dumez	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch (277.93 KB, patch) 2021-05-21 21:38 PDT, Chris Dumez	ews-feeder: commit-queue-	Details Formatted Diff Diff
Patch (277.59 KB, patch) 2021-05-22 08:30 PDT, Chris Dumez	no flags	Details Formatted Diff Diff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.

Chris Dumez

Comment 1 2021-05-21 20:12:26 PDT

Created attachment 429393 [details] Patch

EWS Watchlist

Comment 2 2021-05-21 20:13:27 PDT

Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

Chris Dumez

Comment 3 2021-05-21 20:20:14 PDT

Created attachment 429396 [details] Patch

Chris Dumez

Comment 4 2021-05-21 21:16:50 PDT

Created attachment 429397 [details] Patch

Chris Dumez

Comment 5 2021-05-21 21:38:11 PDT

Created attachment 429398 [details] Patch

Chris Dumez

Comment 6 2021-05-21 22:49:29 PDT

Comment on attachment 429398 [details] Patch Clearing flags on attachment: 429398 Committed r277913 (238046@main): <https://commits.webkit.org/238046@main>